Fix deprecated Grafana dashboards #2242
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Warning
This is a public repository, ensure not to disclose:
What kind of PR is this?
Required: Mark one of the following that is applicable:
Optional: Mark one or more of the following that are applicable:
Important
Breaking changes should be marked
kind/admin-change
orkind/dev-change
depending on typeCritical security fixes should be marked with
kind/security
What does this PR do / why do we need this PR?
This PR updates Grafana Dashboards containing deprecated panels.
node-mixin
was uncommented from the grafana-dashboards makefile and adjusted and seems to be working fine now, but the latest upstream dashboards still included deprecated panels and were fixed manually. There exists an issue regarding this: prometheus/node_exporter#3046. Also, running themake
script fornode-mixin
pulled down two new dashboards, one which was exclusive for darwin/macOS which I removed manually as I am not sure how to configurejsonnet
to only include certain files. The other dashboard wasNode Exporter / USE Method / Multi-cluster
which I kept in.alertmanager-mixin
did not get upgraded with make script, and can't see issues fixing the panels in the repo https://github.com/grafana/prometheus-alertmanager.prometheus-mixin
did not get upgraded with make script. There exists an issue regarding fixing deprecated panels: prometheus/prometheus#14404thanos-mixin
did not get upgraded with make script. Issue regarding fixing deprecated panels: thanos-io/thanos#7413Also noticed that the fluentd dashboard was missing metrics used in some panels, see this commit for newly added config to get this metric.
Information to reviewers
Not sure if the migration script will be necessary, planning to upgrade Grafana to v11 once this gets merged, which will probably make the script obsolete.
I manually checked individual panels when migrating them that they looked close to the original, some panels needed more adjustments than others. Noticed we have some panels with metrics (besides the fluentd one I added) that I did at least not have in my dev environment, might be worth looking into. Might have missed something as there were a lot of dashboards and panels to go through. 😅
Tested running detect-angular-dashboards after having migrated and it did not detect any deprecated panels.
Checklist
NetworkPolicy Dashboard