From 95367d6edf5e3311abe188bec7530fdbd069ad58 Mon Sep 17 00:00:00 2001
From: Andreas Burger <andreas.burger@sap.com>
Date: Mon, 10 Jun 2024 11:49:18 +0200
Subject: [PATCH] Fix indentation in CCM ServiceMonitor (#974) (#975)

The `honorLabels`, `authorization`, `scheme`, and `tlsConfig` fields are
part of the Endpoint API, not the Spec. This indentation error caused the
prometheus-operator to generate a scrape job configuration that specified
http instead of https (because the scheme field was "missing". This in turn
caused CCM-down alerts to fire because the scrape failed with a 400 error.

Co-authored-by: Jeremy Rickards <9338170+rickardsjp@users.noreply.github.com>
---
 .../templates/servicemonitor.yaml                | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/charts/internal/seed-controlplane/charts/cloud-controller-manager/templates/servicemonitor.yaml b/charts/internal/seed-controlplane/charts/cloud-controller-manager/templates/servicemonitor.yaml
index 7fdd1fac0..b7ec9e8b6 100644
--- a/charts/internal/seed-controlplane/charts/cloud-controller-manager/templates/servicemonitor.yaml
+++ b/charts/internal/seed-controlplane/charts/cloud-controller-manager/templates/servicemonitor.yaml
@@ -21,12 +21,12 @@ spec:
       - __name__
       action: keep
       regex: ^(rest_client_requests_total|process_max_fds|process_open_fds)$
-  honorLabels: false
-  authorization:
-    credentials:
-      name: shoot-access-prometheus-shoot
-      key: token
-  scheme: https
-  tlsConfig:
-    insecureSkipVerify: true
+    honorLabels: false
+    authorization:
+      credentials:
+        name: shoot-access-prometheus-shoot
+        key: token
+    scheme: https
+    tlsConfig:
+      insecureSkipVerify: true
 {{- end }}