Skip to content

Bump Grafana Helm chart to v9.3.2#346

Merged
klesh merged 3 commits into
apache:mainfrom
jekader:main
Aug 22, 2025
Merged

Bump Grafana Helm chart to v9.3.2#346
klesh merged 3 commits into
apache:mainfrom
jekader:main

Conversation

@jekader
Copy link
Copy Markdown
Contributor

@jekader jekader commented Aug 7, 2025

The Helm chart used to deploy Grafana is outdated and missing useful features and bugfixes. One of the features usefult for DevLake is the ability to mount configmaps - useful when overriding dashboard configs.

Updating it to latest stable version 9.3.2:
https://github.com/grafana/helm-charts/releases/tag/grafana-9.3.2

Also, the code of the dependency is removed - will be installed during the helm dependency update step and matched against the checksum in Chart.lock.

This fixes #327

@jekader
Copy link
Copy Markdown
Contributor Author

jekader commented Aug 11, 2025

CI is failing due to a timeout when waiting for the Grafana PVC to get claimed:

client.go:144: 2025-08-11 03:03:00 [debug] creating 19 resource(s)
wait.go:50: 2025-08-11 03:03:00 [debug] beginning wait for 19 resources with timeout of 40m0s
ready.go:284: 2025-08-11 03:03:00 [debug] PersistentVolumeClaim is not bound: default/deploy-test-grafana
ready.go:284: 2025-08-11 03:03:02 [debug] PersistentVolumeClaim is not bound: default/deploy-test-grafana
ready.go:284: 2025-08-11 03:03:04 [debug] PersistentVolumeClaim is not bound: default/deploy-test-grafana

the kind cluster does have a storage class:

NAME                 PROVISIONER             RECLAIMPOLICY   VOLUMEBINDINGMODE      ALLOWVOLUMEEXPANSION   AGE
standard (default)   rancher.io/local-path   Delete          WaitForFirstConsumer   false                  28s

Why does it not work? Maybe, the PVC request is larger than the disk on the node? The defaults have not changed across releases so maybe this is some internal bug within the CI setup.

@jekader jekader changed the title Bump Grafana Helm chart to v9.3.1 Bump Grafana Helm chart to v9.3.2 Aug 14, 2025
@jekader jekader marked this pull request as draft August 14, 2025 13:26
@jekader jekader marked this pull request as ready for review August 14, 2025 14:41
@jekader
Copy link
Copy Markdown
Contributor Author

jekader commented Aug 14, 2025

CI should be passing now, issues fixed and tested in my fork

@klesh
Copy link
Copy Markdown
Contributor

klesh commented Aug 18, 2025

@matrixji @JorgeGar Can you help reviewing this PR? Thanks.

Copy link
Copy Markdown
Contributor

@klesh klesh left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution.
Let's move things forward

@klesh klesh merged commit c704ffd into apache:main Aug 22, 2025
3 checks passed
fabioluciano pushed a commit to fabioluciano/incubator-devlake-helm-chart that referenced this pull request May 6, 2026
* bump Grafana helm chart to v9.3.1

* bump Grafana chart to 9.3.2

* update grafana custom image definition
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.

Update Grafana

2 participants