This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new bbf5b8fdf71 Helm chart: start Redis directly, not via shell (#58790)
bbf5b8fdf71 is described below

commit bbf5b8fdf71bc51763ef21b01e408f533cec1296
Author: Ariel Mordoch <[email protected]>
AuthorDate: Fri Nov 28 16:02:56 2025 -0500

    Helm chart: start Redis directly, not via shell (#58790)
---
 chart/templates/redis/redis-statefulset.yaml | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/chart/templates/redis/redis-statefulset.yaml 
b/chart/templates/redis/redis-statefulset.yaml
index ef0d7a38d45..1d23401335c 100644
--- a/chart/templates/redis/redis-statefulset.yaml
+++ b/chart/templates/redis/redis-statefulset.yaml
@@ -92,9 +92,11 @@ spec:
           {{- if $containerLifecycleHooks }}
           lifecycle: {{- tpl (toYaml $containerLifecycleHooks) . | nindent 12 
}}
           {{- end }}
-          command: ["/bin/sh"]
+          command: ["redis-server"]
           resources: {{- toYaml .Values.redis.resources | nindent 12 }}
-          args: ["-c", "exec redis-server --requirepass ${REDIS_PASSWORD}"]
+          # Not launching via shell -- parentheses are required
+          # 
https://kubernetes.io/docs/tasks/inject-data-application/define-command-argument-container/#use-environment-variables-to-define-arguments
+          args: ["--requirepass", "$(REDIS_PASSWORD)"]
           ports:
             - name: redis-db
               containerPort: {{ .Values.ports.redisDB }}

Reply via email to