-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
New dashboard for NGINX #10540
New dashboard for NGINX #10540
Conversation
94c6ed2
to
f6a884b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good for docs!
4128a10
to
290445d
Compare
The |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we should introduce a new dashboard file here and instead we should look into replacing the contents of an existing dashboard.
5c16c25
to
1738aed
Compare
Co-authored-by: Sarah Witt <sarah.witt@datadoghq.com>
Co-authored-by: Sarah Witt <sarah.witt@datadoghq.com>
The |
* Update NGINX dashboard * Update nginx/assets/dashboards/NGINX-Overview_dashboard.json Co-authored-by: Sarah Witt <sarah.witt@datadoghq.com> * Apply suggestions from code review Co-authored-by: Sarah Witt <sarah.witt@datadoghq.com> * Add id from original dashboard * Update id from original dashboard * Change ID to 21 Co-authored-by: Sarah Witt <sarah.witt@datadoghq.com>
What does this PR do?
Updated dashboard for NGINX (See #10522)
Motivation
Additional Notes
Review checklist (to be filled by reviewers)
changelog/
andintegration/
labels attached