Merge pull request #451 from mjdazz/main

Only set REDIS_HOST_PASSWORD when Redis auth is enabled
nextcloud · Oct 15, 2023 · 13a08df · 13a08df
2 parents 2b10d90 + 2abc3dd
commit 13a08df
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/charts/nextcloud/Chart.yaml b/charts/nextcloud/Chart.yaml
@@ -1,6 +1,6 @@
 apiVersion: v2
 name: nextcloud
-version: 4.3.3
+version: 4.3.4
 appVersion: 27.1.2
 description: A file sharing server that puts the control and security of your own data back into your hands.
 keywords:

diff --git a/charts/nextcloud/templates/_helpers.tpl b/charts/nextcloud/templates/_helpers.tpl
@@ -217,6 +217,7 @@ Create environment variables used to configure the nextcloud container as well a
   value: {{ template "nextcloud.redis.fullname" . }}-master
 - name: REDIS_HOST_PORT
   value: {{ .Values.redis.master.service.ports.redis | quote }}
+{{- if .Values.redis.auth.enabled }}
 {{- if and .Values.redis.auth.existingSecret .Values.redis.auth.existingSecretPasswordKey }}
 - name: REDIS_HOST_PASSWORD
   valueFrom:
@@ -228,6 +229,7 @@ Create environment variables used to configure the nextcloud container as well a
   value: {{ .Values.redis.auth.password }}
 {{- end }}
 {{- end }}
+{{- end }}
 {{- if .Values.nextcloud.extraEnv }}
 {{ toYaml .Values.nextcloud.extraEnv }}
 {{- end }}