Fix rbac roles for metrics

Skyscanner · May 31, 2022 · ef75d6a · ef75d6a
1 parent 3859a62
commit ef75d6a
Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/charts/kms-issuer/templates/deployment.yaml b/charts/kms-issuer/templates/deployment.yaml
@@ -34,7 +34,7 @@ spec:
         - containerPort: 8443
           name: https
           protocol: TCP
-      {{- endif }}
+      {{- end }}
       - command:
         - /manager
         - --health-probe-bind-address={{ .Values.args.healthProbeBindAddress }}

diff --git a/charts/kms-issuer/templates/rbac.yaml b/charts/kms-issuer/templates/rbac.yaml
@@ -160,6 +160,7 @@ subjects:
 - kind: ServiceAccount
   name: cert-manager
   namespace: cert-manager
+{{- if .Values.enableMetricsRbacProxy }}
 ---
 apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRole
@@ -171,7 +172,6 @@ rules:
   verbs:
   - get
 ---
-{{- if .Values.enableMetricsRbacProxy }}
 apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRole
 metadata: