Links to Grafana dashboards from MetalK8s UI are broken #3475
Assignees
Labels
kind:bug
Something isn't working
topic:monitoring
Everything related to monitoring of services in a running cluster
topic:ui
UI-related issues
Comments
gdemonet
added
kind:bug
|
Relates to #2820 😕 |
gdemonet
added a commit
that referenced
this issue
Aug 5, 2021
The dashboards we provision with MetalK8s have (or should have, which is why some of this commit's changes are setting some UIDs) a UID. Starting with Grafana v8.0, linking dashboards using their slug is not available anymore (see https://grafana.com/docs/grafana/latest/release-notes/release-notes-8-0-0/#breaking-changes ). To make sure we can use these UIDs safely in links (mostly from MetalK8s UI), we start by listing all of them in a JSON file, and enforce that they are correct when rendering Helm charts. Verifying the final deployed dashboards will be done in a subsequent commit, through a post-install E2E test. See: #3475
gdemonet
added a commit
that referenced
this issue
Aug 5, 2021
We add a test which compares the deployed Grafana dashboards with what is declared in `charts/grafana_dashboard_uids.json`. It will detect: - missing and extra dashboards - UID mismatches This will ensure we can trust the listing in this file, and use it for documenation and/or UI links. See: #3475
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Component: grafana, ui
What happened:
In MetalK8s 2.10, we bumped Grafana to v8.x. However, this broke the links we were using from MetalK8s UI, which used the "by slug" URL format, which have been deprecated since Grafana v5.0.
In Grafana v8.0, this link format was removed entirely: https://grafana.com/docs/grafana/latest/release-notes/release-notes-8-0-0/#breaking-changes
Resolution proposal:
The text was updated successfully, but these errors were encountered: