Merge pull request #4329 from jwenz723/patch-1

[Helm Chart] Add optional service annotations

deploy/charts/cert-manager/templates/service.yaml

@@ -4,6 +4,10 @@ kind: Service
 metadata:
   name: {{ template "cert-manager.fullname" . }}
   namespace: {{ .Release.Namespace | quote }}
+{{- with .Values.serviceAnnotations }}
+  annotations:
+{{ toYaml . | indent 4 }}
+{{- end }}
   labels:
     app: {{ include "cert-manager.name" . }}
     app.kubernetes.io/name: {{ include "cert-manager.name" . }}

deploy/charts/cert-manager/templates/webhook-service.yaml

@@ -3,6 +3,10 @@ kind: Service
 metadata:
   name: {{ template "webhook.fullname" . }}
   namespace: {{ .Release.Namespace | quote }}
+{{- with .Values.webhook.serviceAnnotations }}
+  annotations:
+{{ toYaml . | indent 4 }}
+{{- end }}
   labels:
     app: {{ include "webhook.name" . }}
     app.kubernetes.io/name: {{ include "webhook.name" . }}

deploy/charts/cert-manager/values.yaml

@@ -138,6 +138,9 @@ volumeMounts: []
 
 podLabels: {}
 
+# Optional annotations to add to the controller Service
+# serviceAnnotations: {}
+
 # Optional additional labels to add to the controller Service
 # serviceLabels: {}
 
@@ -248,6 +251,9 @@ webhook:
   # Optional additional annotations to add to the webhook Pods
   # podAnnotations: {}
 
+  # Optional additional annotations to add to the webhook Service
+  # serviceAnnotations: {}
+
   # Optional additional annotations to add to the webhook MutatingWebhookConfiguration
   # mutatingWebhookConfigurationAnnotations: {}