fix(helm-chart): remove double templating of annotations (#2770)

Signed-off-by: RealAnna <anna.reale@dynatrace.com>
Signed-off-by: RealAnna <89971034+RealAnna@users.noreply.github.com>

.github/scripts/.helm-tests/certificates-only/result.yaml

@@ -5,6 +5,8 @@ kind: ServiceAccount
 metadata:
   name: certificate-operator
   namespace: "helmtests"
+  annotations:
+    ciao: commonannotation
   labels:
     app.kubernetes.io/component: rbac
     app.kubernetes.io/part-of: keptn
@@ -14,15 +16,15 @@ metadata:
     app.kubernetes.io/version: vmyversion
     helm.sh/chart: cert-manager-0.2.0
     mamma: test
-  annotations:
-    ciao: commonannotation
 ---
 # Source: keptn/charts/certManager/templates/certificate-operator-rbac.yaml
 apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRole
 metadata:
   name: certificate-operator-role
   namespace: "helmtests"
+  annotations:
+    ciao: commonannotation
   labels:
     app.kubernetes.io/instance: keptn-test
     app.kubernetes.io/managed-by: Helm
@@ -76,6 +78,8 @@ kind: ClusterRoleBinding
 metadata:
   name: certificate-operator-rolebinding
   namespace: "helmtests"
+  annotations:
+    ciao: commonannotation
   labels:
     app.kubernetes.io/component: rbac
     app.kubernetes.io/created-by: certificate-operator
@@ -101,15 +105,15 @@ kind: Role
 metadata:
   name: certificate-operator-role
   namespace: "helmtests"
+  annotations:
+    ciao: commonannotation
   labels:
     app.kubernetes.io/instance: keptn-test
     app.kubernetes.io/managed-by: Helm
     app.kubernetes.io/name: cert-manager
     app.kubernetes.io/version: vmyversion
     helm.sh/chart: cert-manager-0.2.0
     mamma: test
-  annotations:
-    ciao: commonannotation
 rules:
 - apiGroups:
   - ""
@@ -136,6 +140,8 @@ kind: RoleBinding
 metadata:
   name: certificate-operator-leader-election-rolebinding
   namespace: "helmtests"
+  annotations:
+    ciao: commonannotation
   labels:
     app.kubernetes.io/component: rbac
     app.kubernetes.io/created-by: certificate-operator
@@ -146,7 +152,6 @@ metadata:
     app.kubernetes.io/version: vmyversion
     helm.sh/chart: cert-manager-0.2.0
     mamma: test
-  annotations:
     ciao: commonannotation
 roleRef:
   apiGroup: rbac.authorization.k8s.io
@@ -163,6 +168,8 @@ kind: RoleBinding
 metadata:
   name: certificate-operator-role-binding
   namespace: "helmtests"
+  annotations:
+    ciao: commonannotation
   labels:
     app.kubernetes.io/component: rbac
     app.kubernetes.io/created-by: certificate-operator
@@ -173,8 +180,6 @@ metadata:
     app.kubernetes.io/version: vmyversion
     helm.sh/chart: cert-manager-0.2.0
     mamma: test
-  annotations:
-    ciao: commonannotation
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: Role
@@ -190,6 +195,8 @@ kind: Deployment
 metadata:
   name: certificate-operator
   namespace: "helmtests"
+  annotations:
+    ciao: commonannotation
   labels:
     app.kubernetes.io/component: keptn-cert-manager
     app.kubernetes.io/part-of: keptn
@@ -200,8 +207,6 @@ metadata:
     app.kubernetes.io/version: vmyversion
     helm.sh/chart: cert-manager-0.2.0
     mamma: test
-  annotations:
-    ciao: commonannotation
 spec:
   replicas: 1
   selector:

.github/scripts/.helm-tests/default/result.yaml

@@ -11730,6 +11730,11 @@ metadata:
   name: lifecycle-webhook-service
   namespace: "helmtests"
   labels:
+    app.kubernetes.io/instance: keptn-test
+    app.kubernetes.io/managed-by: Helm
+    app.kubernetes.io/name: lifecycle-operator
+    app.kubernetes.io/version: vmyversion
+    helm.sh/chart: lifecycle-operator-0.1.0
 spec:
   type: ClusterIP
   selector:

.github/scripts/.helm-tests/lifecycle-only/result.yaml

@@ -9357,6 +9357,11 @@ metadata:
   name: lifecycle-webhook-service
   namespace: "helmtests"
   labels:
+    app.kubernetes.io/instance: keptn-test
+    app.kubernetes.io/managed-by: Helm
+    app.kubernetes.io/name: lifecycle-operator
+    app.kubernetes.io/version: vmyversion
+    helm.sh/chart: lifecycle-operator-0.1.0
 spec:
   type: ClusterIP
   selector: