Fix wront cluster_id in helm chart

castai · Sep 24, 2024 · a21284c · a21284c
1 parent 24b7181
commit a21284c
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
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 }}
diff --git a/internal/castai/client.go b/internal/castai/client.go
@@ -57,6 +57,13 @@ type client struct {
 
 func NewClient(cfg Config, log logrus.FieldLogger, restyClient *resty.Client, version string) Client {
 	restyClient.BaseURL = cfg.URL
+
+	fmt.Println("Client Config")
+	fmt.Println(fmt.Sprintf("%s%s", userAgent, version))
+	fmt.Println(fmt.Sprintf("%s", cfg.APIKey))
+	fmt.Println(fmt.Sprintf("%s", cfg.ClusterID))
+	fmt.Println(fmt.Sprintf("%s", cfg.URL))
+
 	restyClient.SetHeaders(map[string]string{
 		tokenHeader:           cfg.APIKey,
 		contentTypeHeader:     contentType,

diff --git a/internal/exporter/exporter.go b/internal/exporter/exporter.go
@@ -106,6 +106,8 @@ func (e *exporter) getDCGMUrls(ctx context.Context) ([]string, error) {
 		fieldSelector = fmt.Sprintf("%s,spec.nodeName=%s", fieldSelector, e.cfg.NodeName)
 	}
 
+	e.log.Infof("getting DCGM exporter pods with field selector: %s", fieldSelector)
+
 	// TODO: consider using an informer and keeping a list of pods which match the selector
 	// at the moment seems like an overkill
 	dcgmExporterList, err := e.kube.CoreV1().Pods("").List(ctx, metav1.ListOptions{
@@ -122,6 +124,9 @@ func (e *exporter) getDCGMUrls(ctx context.Context) ([]string, error) {
 		urls[i] = fmt.Sprintf("http://%s:%d%s", dcgmExporter.Status.PodIP, e.cfg.DCGMExporterPort, e.cfg.DCGMExporterPath)
 	}
 
+	e.log.Infof("found %d DCGM exporter pods", len(urls))
+	e.log.Debugf("DCGM exporter urls: %v", urls)
+
 	return urls, nil
 }