fix(monitoring): add cluster labeling to single binary mode

intergral · Oct 10, 2023 · 7bfa5e8 · 7bfa5e8
1 parent 1785839
commit 7bfa5e8
Show file tree

Hide file tree

Showing 3 changed files with 49 additions and 3 deletions.
diff --git a/charts/deep-distributed/templates/lib/service-monitor.yaml b/charts/deep-distributed/templates/lib/service-monitor.yaml
@@ -52,7 +52,7 @@ spec:
           targetLabel: job
         {{- if kindIs "string" .clusterLabel }}
         - replacement: "{{ include "deep.clusterName" $.ctx}}"
-          targetLabel: cluster
+          targetLabel: "{{ .clusterLabel }}"
         {{- end }}
         {{- with .relabelings }}
         {{- toYaml . | nindent 8 }}

diff --git a/charts/deep/templates/serviceMonitor.yaml b/charts/deep/templates/serviceMonitor.yaml
@@ -43,8 +43,8 @@ spec:
           replacement: "{{ include "deep.fullname" $ }}"
           targetLabel: job
         {{- if kindIs "string" .clusterLabel }}
-        - replacement: "{{ include "deep.clusterName" $.ctx}}"
-          targetLabel: cluster
+        - replacement: "{{ include "deep.clusterName" $ }}"
+          targetLabel: "{{ .clusterLabel }}"
         {{- end }}
         {{- with .relabelings }}
         {{- toYaml . | nindent 8 }}

diff --git a/charts/deep/unittests/serviceMonitor_test.yaml b/charts/deep/unittests/serviceMonitor_test.yaml
@@ -0,0 +1,46 @@
+suite: test service monitor
+chart:
+  version: 9.9.9+test
+  appVersion: 1.1.1+test
+release:
+  name: deep
+set:
+  monitoring:
+    enabled: true
+    serviceMonitor:
+      namespace: monitoring
+      labels:
+        release: prometheus
+tests:
+  - it: should create correct servicemonitor
+    template: serviceMonitor.yaml
+    asserts:
+      - isKind:
+          of: ServiceMonitor
+      - matchRegex:
+          path: metadata.name
+          pattern: deep
+  - it: should have cluster label
+    template: serviceMonitor.yaml
+    asserts:
+      - equal:
+          path: spec.endpoints[0].relabelings[1].replacement
+          value: deep
+      - equal:
+          path: spec.endpoints[0].relabelings[1].targetLabel
+          value: cluster
+  - it: cluster label should use config
+    set:
+      config:
+        clusterName: "fromconfig"
+      monitoring:
+        serviceMonitor:
+          clusterLabel: "labelfromconfig"
+    template: serviceMonitor.yaml
+    asserts:
+      - equal:
+          path: spec.endpoints[0].relabelings[1].replacement
+          value: fromconfig
+      - equal:
+          path: spec.endpoints[0].relabelings[1].targetLabel
+          value: labelfromconfig