charts/cannon: fix syntax for manually setting tls certificate (#2558)

* charts/cannon: fix syntax for manually setting tls certificate * Hi CI
wireapp · Jul 14, 2022 · 3f8ebd7 · 3f8ebd7
1 parent 250728a
commit 3f8ebd7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/changelog.d/3-bug-fixes/cannon-nginz-custom-tls b/changelog.d/3-bug-fixes/cannon-nginz-custom-tls
@@ -0,0 +1 @@
+Fix a bug in charts/cannon. It's now possible to use a custom TLS certificate when enabling cannon's nginz sidecar container. (Previously only letsencrypt certificates worked, and were tested)
diff --git a/charts/cannon/templates/nginz-certificate-secret.yaml b/charts/cannon/templates/nginz-certificate-secret.yaml
@@ -10,6 +10,6 @@ metadata:
     heritage: "{{ .Release.Service }}"
 type: kubernetes.io/tls
 data:
-  tls.crt: {{ .Values.secrets.nginz.tls.crt }}
-  tls.key: {{ .Values.secrets.nginz.tls.key }}
+  tls.crt: {{ .Values.secrets.nginz.tls.crt | b64enc | quote }}
+  tls.key: {{ .Values.secrets.nginz.tls.key | b64enc | quote }}
 {{- end }}
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Fix a bug in charts/cannon. It's now possible to use a custom TLS certificate when enabling cannon's nginz sidecar container. (Previously only letsencrypt certificates worked, and were tested)