Skip to content
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

Dashboards: use timeseries panel to show exemplars #2800

Merged
merged 3 commits into from
Aug 26, 2022

Conversation

bboreham
Copy link
Contributor

@bboreham bboreham commented Aug 22, 2022

This is now "the default and primary way to visualize time series data"; the 'graph' panel has been deprecated.

Uses upstream change grafana/jsonnet-libs#824

So far I have only changed the per-pod latency panels, as those are the ones where exemplars work.

Checklist

  • NA Tests updated
  • NA Documentation added
  • CHANGELOG.md updated

Copy link
Contributor

@dimitarvdimitrov dimitarvdimitrov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried importing the remote ruler reads dashboard. I can still see the legend though 🤔 below is the current and previous panel
Screenshot 2022-08-24 at 12 04 45
Screenshot 2022-08-24 at 12 05 22

@pracucci
Copy link
Collaborator

I tried importing the remote ruler reads dashboard. I can still see the legend though 🤔 below is the current and previous panel

I also did some comparison and this is the only difference I noticed (except exemplars, as expected). This is very curious because looks like the "showLegend": false is ignored and switches to true when the .json is loaded by Grafana. Is this a Grafana bug? 🤔

@bboreham
Copy link
Contributor Author

Thanks for reviews.

I can still see the legend though
Is this a Grafana bug?

I spent a lot of time tweaking parameters to see if I could make the legend go away.
Certainly seems an unexpected behaviour of Grafana.
To report as a bug I need some way to repro; one way would be to merge this ;-)

Copy link
Collaborator

@pracucci pracucci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, to let you merge it! Do you mind opening an issue in Mimir too, so we don't forget about it, please? :)

This is now "the default and primary way to visualize time series data":
https://grafana.com/docs/grafana/latest/visualizations/time-series/

Uses upstream change grafana/jsonnet-libs#824

So far I have only changed the per-pod latency panels, as those are the
ones where exemplars work.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants