fix: ensure let's encrypt email can be set

Signed-off-by: Gabor Boros <[email protected]>
openedx · Dec 17, 2024 · a7a3b2f · a7a3b2f
1 parent 3b9be99
commit a7a3b2f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/charts/harmony-chart/templates/issuer.yaml b/charts/harmony-chart/templates/issuer.yaml
@@ -5,8 +5,8 @@ metadata:
   name: harmony-letsencrypt-global
 spec:
   acme:
-    {{- if index .Values "cert-manager" "email" }}
-    email: {{ index .Values "cert-manager" "email" }}
+    {{- if .Values.notificationEmail }}
+    email: {{ .Values.notificationEmail }}
     {{- end }}
     privateKeySecretRef:
       name: harmony-letsencrypt-global

diff --git a/charts/harmony-chart/values.yaml b/charts/harmony-chart/values.yaml
@@ -4,6 +4,10 @@
 
 clusterDomain: "*"
 
+# Email address used for receiving Let's encrypt notifications if cert-manager
+# is enabled.
+notificationEmail: null
+
 ingress-nginx:
   # Use ingress-nginx as a default controller.
   enabled: true