fix(alerts): refine chart test (#660)

* fix(alerts): rename CA secret

Signed-off-by: Richard Tief <richard.tief@sap.com>

* fix(alerts): add ticks to test resources

Signed-off-by: Richard Tief <richard.tief@sap.com>

* chore(alerts): split tests

Signed-off-by: Richard Tief <richard.tief@sap.com>

* chore(alerts): add test description

Signed-off-by: Richard Tief <richard.tief@sap.com>

* chore(alerts): use regular expression for exact match of a resource

Signed-off-by: Richard Tief <richard.tief@sap.com>

---------

Signed-off-by: Richard Tief <richard.tief@sap.com>

Author: richardtief

alerts/charts/Chart.yaml

@@ -8,7 +8,7 @@ maintainers:
 name: alerts
 sources:
   - https://github.com/cloudoperators/greenhouse-extensions
-version: 0.19.7
+version: 0.19.8
 keywords:
   - prometheus-alertmanager
 dependencies:

alerts/charts/templates/certmanager.yaml

@@ -19,7 +19,7 @@ metadata:
   labels:
 {{- include "kube-prometheus-stack.labels" . | indent 4 }}
 spec:
-  secretName: tls-alerts-{{ .Release.Namespace  }}-root
+  secretName: tls-alerts-{{ .Release.Namespace  }}-root-ca
   duration: {{ .Values.alerts.certManager.rootCert.duration | default "43800h0m0s" | quote }}
   issuerRef:
     name: {{ .Release.Namespace }}-alerts-issuer

alerts/charts/templates/tests/test-alerts-config.yaml

@@ -46,17 +46,22 @@ data:
     }
     {{- end }}
 
-    {{- if and .Values.alerts.alertmanager.ingress.enabled .Values.alerts.alertmanager.ingress.hosts .Values.alerts.certManager.enabled }}
-    @test "Generated server certificate for the Prometheus Alertmanager to use" {
-        verify "there is 1 issuer named '{{ .Release.Namespace }}-alerts-issuer'"
-        verify "there is 1 issuer named '{{ .Release.Namespace }}-alerts-root-issuer"
-        verify "there is 1 certificate named '{{ .Release.Namespace }}-alerts-root-cert'"
-        verify "there is 1 certificate named 'alertmanager-{{ .Release.Namespace }}-cert'"
-        verify "there is 1 certificate named 'prometheus-{{ .Release.Namespace }}-cert'"
-        verify "there is 1 secret named 'tls-alertmanager-{{ .Release.Namespace }}'"
-        verify "there is 1 secret named 'tls-prometheus-{{ .Release.Namespace }}"
+    {{- if .Values.alerts.certManager.enabled }}
+    @test "Generated server certificates for the Prometheus Alertmanager to use" {
+        verify "there is 1 issuer named '^{{ .Release.Namespace }}-alerts-issuer$'"
+        verify "there is 1 issuer named '^{{ .Release.Namespace }}-alerts-root-issuer$'"
+        verify "there is 1 certificate named '^{{ .Release.Namespace }}-alerts-root-cert$'"
+        verify "there is 1 certificate named '^alertmanager-{{ .Release.Namespace }}-cert$'"
+        verify "there is 1 certificate named '^prometheus-{{ .Release.Namespace }}-cert$'"
+        verify "there is 1 secret named '^tls-alertmanager-{{ .Release.Namespace }}$'"
+        verify "there is 1 secret named '^tls-prometheus-{{ .Release.Namespace }}$'"
+    }
+    {{- end }}
 
+    {{- if  and .Values.alerts.alertmanager.ingress.enabled .Values.alerts.alertmanager.ingress.hosts .Values.alerts.certManager.enabled }}
+    @test "Verify the availability of the Alertmanager API by using the certificates created" {
         url="https://{{ first .Values.alerts.alertmanager.ingress.hosts }}/-/healthy"
+
         run curl --cert /tls-assets/tls.crt --key /tls-assets/tls.key ${url}
         [ "$status" -eq "0" ]
     }

alerts/plugindefinition.yaml

@@ -6,7 +6,7 @@ kind: PluginDefinition
 metadata:
   name: alerts
 spec:
-  version: 2.7.7
+  version: 2.7.8
   weight: 0
   displayName: Alerts
   description: The Alerts Plugin consists of both Prometheus Alertmanager and Supernova, the holistic alert management UI
@@ -15,7 +15,7 @@ spec:
   helmChart:
     name: alerts
     repository: oci://ghcr.io/cloudoperators/greenhouse-extensions/charts
-    version: 0.19.7
+    version: 0.19.8
   uiApplication:
     name: supernova
     version: "latest"