Merge pull request #36 from castai/fix/cluster-id-in-helm-chart

Fix wront cluster_id in helm chart
castai · Sep 24, 2024 · c160e5f · c160e5f
2 parents 24b7181 + 3b619ea
commit c160e5f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/charts/gpu-metrics-exporter/templates/daemonset.yaml b/charts/gpu-metrics-exporter/templates/daemonset.yaml
@@ -74,19 +74,19 @@ spec:
           envFrom:
             - configMapRef:
                 name: {{- include "gpu-metrics-exporter.config-map" . | indent 1}}
-          {{- if .Values.dcgmExporter.enabled }}
           env:
-            - name: "DCGM_HOST"
-              value: "localhost"
             - name: "NODE_NAME"
               valueFrom:
                 fieldRef:
                   fieldPath: spec.nodeName
             - name: "API_KEY"
               valueFrom:
-                  secretKeyRef:
-                    name: {{ include "gpu-metrics-exporter.fullname" . }}
-                    key: API_KEY
+                secretKeyRef:
+                  name: {{ include "gpu-metrics-exporter.fullname" . }}
+                  key: API_KEY
+          {{- if .Values.dcgmExporter.enabled }}
+            - name: "DCGM_HOST"
+              value: "localhost"
           {{- end }}
           resources:
             {{- toYaml .Values.gpuMetricsExporter.resources | nindent 12 }}

diff --git a/charts/gpu-metrics-exporter/templates/gpu-exporter-configmap.yaml b/charts/gpu-metrics-exporter/templates/gpu-exporter-configmap.yaml
@@ -4,6 +4,6 @@ metadata:
   name: {{- include "gpu-metrics-exporter.config-map" . | indent 1}}
 data:
 {{- with .Values.gpuMetricsExporter.config }}
-  CLUSTER_ID: "{{ .Values.castai.clusterId | default .CLUSTER_ID }}"
+  CLUSTER_ID: "{{ $.Values.castai.clusterId | default .CLUSTER_ID }}"
   {{- toYaml (omit . "CLUSTER_ID") | nindent 2 }}
 {{- end }}