From 5931965861f01f4be324c0d36427a1cb3c7353d8 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Tue, 1 Jul 2025 20:04:46 +0000 Subject: [PATCH] fix: Correct Helm chart label in Grafana dashboard ConfigMap Updates the `helm.sh/chart` label in the Grafana dashboard ConfigMap to use `{{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}`. This resolves a Helm linting error caused by an incorrect template reference. The previous commit added the Grafana dashboard as a ConfigMap: feat: Add Grafana dashboard as ConfigMap Adds the Grafana dashboard for iperf3-monitor as a ConfigMap to the Helm chart. The dashboard is sourced from the project's README and stored in `charts/iperf3-monitor/grafana/iperf3-dashboard.json`. A new template `charts/iperf3-monitor/templates/grafana-dashboard-configmap.yaml` creates the ConfigMap, loading the dashboard JSON and labeling it with `grafana_dashboard: "1"` to enable auto-discovery by Grafana. --- .../iperf3-monitor/templates/grafana-dashboard-configmap.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/charts/iperf3-monitor/templates/grafana-dashboard-configmap.yaml b/charts/iperf3-monitor/templates/grafana-dashboard-configmap.yaml index 7403ad3..f28b429 100644 --- a/charts/iperf3-monitor/templates/grafana-dashboard-configmap.yaml +++ b/charts/iperf3-monitor/templates/grafana-dashboard-configmap.yaml @@ -5,7 +5,7 @@ metadata: labels: grafana_dashboard: "1" app.kubernetes.io/name: {{ include "iperf3-monitor.name" . }} - helm.sh/chart: {{ include "iperf3-monitor.chart" . }} + helm.sh/chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }} app.kubernetes.io/instance: {{ .Release.Name }} app.kubernetes.io/managed-by: {{ .Release.Service }} data: