Fix servicemonitor relabeling

Signed-off-by: Alexey Kubrinsky <akubrinsky@zetaglobal.com>
kedacore · Jun 6, 2023 · 0bd1bc1 · 0bd1bc1
1 parent c5fad71
commit 0bd1bc1
Show file tree

Hide file tree

Showing 4 changed files with 29 additions and 14 deletions.
diff --git a/.github/workflows/ci-core.yml b/.github/workflows/ci-core.yml
@@ -64,7 +64,7 @@ jobs:
 
     - name: Helm install
       uses: Azure/setup-helm@v3
-      
+
     - name: Create k8s ${{ matrix.kubernetesVersion }} Kind Cluster
       uses: helm/kind-action@main
       with:
@@ -88,9 +88,9 @@ jobs:
         image:
           keda:
             tag: main
-          metricsApiServer:   
+          metricsApiServer:
             tag: main
-          webhooks:   
+          webhooks:
             tag: main
         podIdentity:
           azureWorkload:
@@ -99,7 +99,7 @@ jobs:
             clientId: ${{ matrix.clientId }}
         podDisruptionBudget:
           operator:
-            maxUnavailable: 1 
+            maxUnavailable: 1
           metricServer:
             maxUnavailable: 1
           webhooks:
@@ -109,10 +109,25 @@ jobs:
             enabled: true
             podMonitor:
               enabled: true
+            serviceMonitor:
+              enabled: true
+              relabellings:
+              - regex: (go_.*)
+                action: drop
           webhooks:
             enabled: true
             serviceMonitor:
               enabled: true
+              relabellings:
+              - regex: (go_.*)
+                action: drop
+          metricServer:
+            enabled: true
+            serviceMonitor:
+              enabled: true
+              relabellings:
+              - regex: (go_.*)
+                action: drop
         webhooks:
           failurePolicy: Fail
         certificates:

diff --git a/keda/templates/manager/servicemonitor.yaml b/keda/templates/manager/servicemonitor.yaml
@@ -20,11 +20,11 @@ spec:
   {{- end }}
   {{- with .Values.prometheus.operator.serviceMonitor.targetLabels }}
   targetLabels:
-  {{ toYaml . | indent 4 }}
+  {{- toYaml . | nindent 4 }}
   {{- end }}
   {{- with .Values.prometheus.operator.serviceMonitor.podTargetLabels }}
   podTargetLabels:
-  {{ toYaml . | indent 4 }}
+  {{- toYaml . | nindent 4 }}
   {{- end }}
   endpoints:
   - port: {{ .Values.prometheus.operator.serviceMonitor.port }}
@@ -40,7 +40,7 @@ spec:
     {{- end }}
     {{- with .Values.prometheus.operator.serviceMonitor.relabellings }}
     relabelings:
-    {{ toYaml . | indent 6 }}
+    {{- toYaml . | nindent 6 }}
     {{- end }}
   namespaceSelector:
     matchNames:

diff --git a/keda/templates/metrics-server/servicemonitor.yaml b/keda/templates/metrics-server/servicemonitor.yaml
@@ -20,11 +20,11 @@ spec:
   {{- end }}
   {{- with .Values.prometheus.metricServer.serviceMonitor.targetLabels }}
   targetLabels:
-  {{ toYaml . | indent 4 }}
+  {{- toYaml . | nindent 4 }}
   {{- end }}
   {{- with .Values.prometheus.metricServer.serviceMonitor.podTargetLabels }}
   podTargetLabels:
-  {{ toYaml . | indent 4 }}
+  {{- toYaml . | nindent 4 }}
   {{- end }}
   endpoints:
   - port: {{ .Values.prometheus.metricServer.portName }}
@@ -40,7 +40,7 @@ spec:
     {{- end }}
     {{- with .Values.prometheus.metricServer.serviceMonitor.relabellings }}
     relabelings:
-    {{ toYaml . | indent 6 }}
+    {{- toYaml . | nindent 6 }}
     {{- end }}
   namespaceSelector:
     matchNames:

diff --git a/keda/templates/webhooks/servicemonitor.yaml b/keda/templates/webhooks/servicemonitor.yaml
@@ -21,11 +21,11 @@ spec:
   {{- end }}
   {{- with .Values.prometheus.webhooks.serviceMonitor.targetLabels }}
   targetLabels:
-  {{ toYaml . | indent 4 }}
+  {{- toYaml . | nindent 4 }}
   {{- end }}
   {{- with .Values.prometheus.webhooks.serviceMonitor.podTargetLabels }}
   podTargetLabels:
-  {{ toYaml . | indent 4 }}
+  {{- toYaml . | nindent 4 }}
   {{- end }}
   endpoints:
   - port: {{ .Values.prometheus.webhooks.serviceMonitor.port }}
@@ -41,7 +41,7 @@ spec:
     {{- end }}
     {{- with .Values.prometheus.webhooks.serviceMonitor.relabellings }}
     relabelings:
-    {{ toYaml . | indent 6 }}
+    {{- toYaml . | nindent 6 }}
     {{- end }}
   namespaceSelector:
     matchNames:
@@ -50,4 +50,4 @@ spec:
     matchLabels:
       app.kubernetes.io/name: {{ .Values.webhooks.name }}
 {{- end }}
-{{- end }}
+{{- end }}