fix(redis): Remove fullnameOverride from redis

iterative · Jul 13, 2023 · c97cb13 · c97cb13
1 parent 2bd0779
commit c97cb13
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/charts/studio/templates/configmap-studio.yaml b/charts/studio/templates/configmap-studio.yaml
@@ -46,19 +46,19 @@ data:
   {{- if .Values.global.celery.brokerUrl }}
   CELERY_BROKER_URL: {{ .Values.global.celery.brokerUrl }}
   {{- else }}
-  CELERY_BROKER_URL: "redis://{{ .Values.redis.fullnameOverride }}-master.{{ .Release.Namespace }}.svc.cluster.local:6379"
+  CELERY_BROKER_URL: "redis://{{ .Release.Name }}-redis-master.{{ .Release.Namespace }}.svc.cluster.local:6379"
   {{- end }}
 
   {{- if .Values.global.celery.resultBackend }}
   CELERY_RESULT_BACKEND: {{ .Values.global.celery.resultBackend }}
   {{- else }}
-  CELERY_RESULT_BACKEND: "redis://{{ .Values.redis.fullnameOverride }}-master.{{ .Release.Namespace }}.svc.cluster.local:6379"
+  CELERY_RESULT_BACKEND: "redis://{{ .Release.Name }}-redis-master.{{ .Release.Namespace }}.svc.cluster.local:6379"
   {{- end }}
 
   {{- if .Values.global.celery.resultBackend }}
   REDIS_URL: {{ .Values.global.celery.resultBackend }}
   {{- else }}
-  REDIS_URL: "redis://{{ .Values.redis.fullnameOverride }}-master.{{ .Release.Namespace }}.svc.cluster.local:6379"
+  REDIS_URL: "redis://{{ .Release.Name }}-redis-master.{{ .Release.Namespace }}.svc.cluster.local:6379"
   {{- end }}
 
   {{- if .Values.global.scmProviders.github.apiUrl }}

diff --git a/charts/studio/values.yaml b/charts/studio/values.yaml
@@ -250,8 +250,13 @@ nginx:
 redis:
   # -- Redis enabled
   enabled: true
-  # -- Redis name override
-  fullnameOverride: studio-redis
+
+  # -- Redis master configuration
+  master:
+    # -- Redis master persistence configuration
+    persistence:
+      # -- Redis master persistence size in PVC
+      size: "1Gi"
 
   # -- Redis master configuration
   master: