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
PublicDashboards: Dont support exemplars #54933
PublicDashboards: Dont support exemplars #54933
Conversation
…ey were exclusively made for it. When grouping dashboard queries, remove the exemplar property if it exists. We wont be supporting exemplars for pubdash right now since all the exemplar transformation happens within the prometheus datasource on the frontend.
Drone build failed: https://drone.grafana.net/grafana/grafana-enterprise/32767 |
This pull request was removed from the 9.1.4 milestone because 9.1.4 is currently being released. |
nit related to the title: The area should be one word in camel case and the description should start with capital letter. |
Drone build failed: https://drone.grafana.net/grafana/grafana-enterprise/33301 |
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!
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!
Problem
Exemplars are handled within the Prometheus datasource on the frontend. There's a lot of code that handles those transformations, and currently, it's low priority to add exemplar support for our
PublicDashboardDatasource
.Solution
When gathering queries on the backend for public dashboards, we remove the exemplar property if it exists.
Notes
I moved the dashboard_queries into the publicdashboards service because they're only used for public dashboards.
fixes #54035