-
Notifications
You must be signed in to change notification settings - Fork 543
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
Remove "Inflight requests" from object store panels #3521
Conversation
Signed-off-by: Marco Pracucci <marco@pracucci.com>
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.
But we don't really have a place to put such component-specific metrics. I think with the streaming Series() implementation there will be new such metrics.
Should we start having a "Debugging" dashboard that goes into more details?
@dimitarvdimitrov I think "Mimir / Queries" dashboard is a good candidate for such metrics. I would try to fit them there before introducing yet another dashboard. |
Signed-off-by: Marco Pracucci <marco@pracucci.com> Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com> Signed-off-by: Marco Pracucci <marco@pracucci.com> (cherry picked from commit ce49b04)
Signed-off-by: Marco Pracucci <marco@pracucci.com> Signed-off-by: Marco Pracucci <marco@pracucci.com>
…rafana#3573) Signed-off-by: Marco Pracucci <marco@pracucci.com> Signed-off-by: Marco Pracucci <marco@pracucci.com> (cherry picked from commit ce49b04) Co-authored-by: Marco Pracucci <marco@pracucci.com>
What this PR does
This PR reverts #2914. As explained in #3324 (comment), this panel is wrong among object store panels because it queries (a) a store-gateway specific metric which (b) tracks the number of inflight
Series()
requests in the store-gateway and not the number of inflight requests to the store-gateway.Which issue(s) this PR fixes or relates to
N/A
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]