Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add dashboard monitoring resources #54

Merged
merged 5 commits into from
Dec 2, 2021

Conversation

heyealex
Copy link
Contributor

@heyealex heyealex commented Dec 2, 2021

Added a resources that creates a monitoring dashboard based on a
json.tpl file. The dashboards can be customized with widgets that may
eventually be outputs from other resources and can be setup to filter by
deployment_name.

Also includes two default dashboard JSON files:

  • Empty: empty with the exception of a text box
  • HPC: HPC focused dashboard

Submission Checklist:

  • Have you installed and run this change against pre-commit? pre-commit install
  • Are all tests passing? make tests
  • If applicable, have you written additional unit tests to cover this
    change?
  • Is unit test coverage still above 80%?
  • Have you updated any application documentation such as READMEs and user
    guides?
  • Have you followed the guidelines in our Contributing document?

Added a resources that creates a monitoring dashboard based on a
json.tpl file. The dashboards can be customized with widgets that may
eventually be outputs from other resources and can be setup to filter by
deployment_name.

Also includes two default dashboard JSON files:
* Empty: empty with the exception of a text box
* HPC: HPC focused dashboard
Copy link
Member

@cboneti cboneti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see comments

resources/monitoring/dashboard/README.md Outdated Show resolved Hide resolved
resources/monitoring/dashboard/dashboards/Empty.json.tpl Outdated Show resolved Hide resolved
resources/monitoring/dashboard/dashboards/HPC.json.tpl Outdated Show resolved Hide resolved
}

variable "base_dashboard" {
description = "Baseline dashboard template, either custom or from ./dashboards"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so, what are the options here? a path?
does that mean I can add ~/my-dash.json?

@cboneti cboneti self-requested a review December 2, 2021 01:05
@heyealex
Copy link
Contributor Author

heyealex commented Dec 2, 2021

Addressed the feedback in the last commit. I also added "title" as a variable and set the default text boxes to that and the title of the dashboard to "title: deployment_name" to make it more flexible overall.

@heyealex heyealex marked this pull request as ready for review December 2, 2021 01:25
@heyealex heyealex merged commit 06c868d into GoogleCloudPlatform:develop Dec 2, 2021
@heyealex heyealex deleted the monitoring-dashboard branch December 2, 2021 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants