Conversation
markmandel
left a comment
There was a problem hiding this comment.
We should also update the make reference please!
build/docs/make-reference.md
|
Sure @markmandel. I missed that one. |
|
/gcbrun |
|
Build Succeeded 🥳 Build Id: e9e8d3e7-c99b-415f-a907-3e439c0e47b6 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version: |
mikeseese
left a comment
There was a problem hiding this comment.
Just needs this one missed update; I can't see any more references other than in vendor/k8s.io/apimachinery/pkg/util/managedfields/node.yaml which I believe is negligible.
Co-authored-by: Mike Seese <seesemichaelj@gmail.com>
|
@souben apologies for the delay on this one! Totally slipped through the cracks with GDC and the move to CNCF. Can you please sign your commit - since we're now on a DCO system (see the action for details) rather than a CLA. Just taking this for a spin now locally to make sure it all installs appropriately. |
@mikeseese I'm not sure I'm following this comment - do we need to upgrade a dependency for some reason? |
markmandel
left a comment
There was a problem hiding this comment.
Took this for a run! Looks good!
Just need a signing on the git commit please.
|
/gcbrun |
|
Build Succeeded 🥳 Build Id: 0cb0dcb3-81cc-4ed0-bbf3-dbd7dfdfefe5 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version: |
@markmandel I was referencing the collapsed suggested change in my review, which as been resolved:
I believe the |
|
Yeah that's just a vendored dependency - it has no impact on this issue |

What type of PR is this?
/kind cleanup
What this PR does / Why we need it
The Grafana Helm chart currently used by the project has been deprecated.
The chart was migrated to
grafana-community/helm-chartseffective January 30, 2026.Running
make minikube-setup-grafanaproduces a deprecation warning, and this PR is to migrate the project fromgrafana/grafanatografana-community/grafana.Note that a previous PR (#4449) addressing the issue was submitted previously, but it was stalled for around a week due to licencing issue, so this PR was submitted as a backup.
Changes
1. build/Makefile
Updated the
setup-grafanatarget:Old:
New:
The
uninstall-grafanatarget remains unchanged since it references the release name.2. build/grafana.yaml and build/grafana-frontend.yaml
Reviewed for compatibility with the new chart.
Current values work with the upgraded chart. Verified by installing the new chart and confirming dashboards load correctly.
3. Documentation
Updated
site/content/en/docs/Guides/metrics.md:https://grafana-community.github.io/helm-chartshelm repo addandhelm upgradecommands to usegrafana-communityUpdated the
build/docs/make-reference.mdto reference the new Grafana Helm chart.Which issue(s) this PR fixes
Closes #4447