-
Notifications
You must be signed in to change notification settings - Fork 458
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 monitoring for API Server Watches #2743
Conversation
/invite @timebertt |
Thank you very much for the PR! |
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.
Very nice PR!
/invite @istvanballok
charts/seed-monitoring/charts/core/charts/prometheus/rules/kube-apiserver.rules.yaml
Outdated
Show resolved
Hide resolved
charts/seed-monitoring/charts/grafana/dashboards/operators/apiserver-watches-dashboard.json
Outdated
Show resolved
Hide resolved
charts/seed-monitoring/charts/grafana/dashboards/operators/apiserver-watches-dashboard.json
Outdated
Show resolved
Hide resolved
charts/seed-monitoring/charts/grafana/dashboards/operators/apiserver-watches-dashboard.json
Outdated
Show resolved
Hide resolved
charts/seed-monitoring/charts/grafana/dashboards/operators/apiserver-watches-dashboard.json
Outdated
Show resolved
Hide resolved
charts/seed-monitoring/charts/core/charts/prometheus/rules/kube-apiserver.rules.yaml
Outdated
Show resolved
Hide resolved
charts/seed-monitoring/charts/grafana/dashboards/operators/apiserver-watches-dashboard.json
Outdated
Show resolved
Hide resolved
574fb90
to
d2c0bd3
Compare
d2c0bd3
to
e8460f6
Compare
Thanks for the review @timebertt! I've addressed the comments. PTAL |
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.
Thanks! Great work, I like it :)
/lgtm
/retest |
@timebertt Command |
/ok-to-test |
* Add metric shoot:apiserver_watch_duration:quantile * Add dashboard to show length of API Server watches based on resource
e8460f6
to
5eb01db
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.
/lgtm
Kudos! 👍
Added thresholds to the panels to add visual clues for which values are acceptable. |
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.
Good idea!
/lgtm
How to categorize this PR?
/area monitoring
/kind enhancement
/priority normal
What this PR does / why we need it:
Add metric shoot:apiserver_watch_duration:quantile. Add dashboard to show length of API Server watches based on resource. These metrics can be helpful in identifying resources with troublesome WATCHES. WATCHES should last a long time so if a resource has many WATCHES that are very short this could point to an issue.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note: