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
Prometheus: Fix exemplars not respecting corresponding series display status. #59743
Prometheus: Fix exemplars not respecting corresponding series display status. #59743
Conversation
public/app/plugins/panel/timeseries/plugins/ExemplarsPlugin.tsx
Outdated
Show resolved
Hide resolved
Getting closer, although the code is getting messier, but there's still a bug when you're viewing multiple series (by holding shift and clicking), in which only a single label filter is being applied. |
…lars wouldn't filter
The e2e test cases we currently have are only testing a single series, we'll need to revisit our exemplar e2e tests in another issue: #59911. |
…any label name, label value combination
@itsmylife thanks for the feedback, I've implemented a unit test for the exported function. Can you take another look in the morning? |
…678/exemplars-not-filtering-from-legend
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.
Makes sense to me, well done! LGTM!
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 please revert the changes in CHANGELOG.md
file.
…678/exemplars-not-filtering-from-legend
… time series. (#60054) Prometheus: Fix exemplar fill color to match series color in time series. (#59908) * Implementing new optional color property to ExemplarMarker component * Refactor some changes from #59743 (cherry picked from commit dfc1516) Co-authored-by: Galen Kistler <109082771+gtk-grafana@users.noreply.github.com>
…s display status. (grafana#60047) Prometheus: Fix exemplars not respecting corresponding series display status. (grafana#59743) * Exemplar filtering when series are toggled in legend UI (cherry picked from commit 22f8283) Co-authored-by: Galen Kistler <109082771+gtk-grafana@users.noreply.github.com>
… time series. (grafana#60054) Prometheus: Fix exemplar fill color to match series color in time series. (grafana#59908) * Implementing new optional color property to ExemplarMarker component * Refactor some changes from grafana#59743 (cherry picked from commit dfc1516) Co-authored-by: Galen Kistler <109082771+gtk-grafana@users.noreply.github.com>
What is this feature?
Fix bug in which exemplars do not filter when a series is hidden via user clicking on the legend UI.
Currently, when filtering a metric with multiple labels/series via the legend UI, the exemplars do not respect the active filters, and will always display all exemplars in the current time-series graph.
Why do we need this feature?
Users with many labels for a single series in an exemplar query are left with a heap of exemplars that are not differentiated. This PR allows users to hide series from the graph and only view the relevant exemplars.
Who is this feature for?
Users of prometheus exemplars.
Which issue(s) does this PR fix?:
Fixes #59678
Special notes for your reviewer:
This might not be the most clean or efficient solution, please let me know if there's any improvements we can make to this!
Debug snapshot from the TNS app:
histogram_quantile(0.95, sum by(le, job) (rate(tns_request_duration_seconds_bucket[$__rate_interval])))
debug-Exemplar 1 metric 3 labels-2022-12-07 11_53_14.json.txt