-
Notifications
You must be signed in to change notification settings - Fork 153
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
ad docs and dashboard for indexer RED metrics #3892
Conversation
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.
Added a couple of comments.
It would be also great if you could give a screenshot or environment where we see the added panels in action?
@@ -172,7 +172,7 @@ | |||
"uid": "P1809F7CD0C75ACF3" | |||
}, | |||
"editorMode": "code", | |||
"expr": "sum(rate(http_request_duration_seconds_count{handler=\"/v1/query\"}[1m]))", | |||
"expr": "sum(rate(http_request_duration_seconds_count{handler=\"/v1/query\"}[2m]))", |
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.
what would be the context or motivation for the 2m
change? why this is better?
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.
Docs for
weaveworks/weave-gitops-enterprise#3134
What changed?
Collecting:
Explorer monitoring documentation extended with indexer write metrics
Dashboard:
Added indexer writes panels to explorer grafana dashboard
Why was this change made?
Increase observability for datastore and monitor RED metrics for write operations
How did you validate the change?
Manual test
Release notes
added docs for indexer write metrics
Documentation Changes