From 8740a087b9c28bbea3a6b976ef54518fad8a94c6 Mon Sep 17 00:00:00 2001 From: Sheya Bernstein Date: Sun, 11 Jan 2026 20:14:16 +0000 Subject: [PATCH] fix: apply tpl function to all component extraEnvironmentVars (#8001) --- k8s/charts/seaweedfs/templates/admin/admin-statefulset.yaml | 4 ++-- .../seaweedfs/templates/all-in-one/all-in-one-deployment.yaml | 4 ++-- k8s/charts/seaweedfs/templates/cosi/cosi-deployment.yaml | 4 ++-- k8s/charts/seaweedfs/templates/filer/filer-statefulset.yaml | 4 ++-- k8s/charts/seaweedfs/templates/master/master-statefulset.yaml | 4 ++-- k8s/charts/seaweedfs/templates/s3/s3-deployment.yaml | 4 ++-- k8s/charts/seaweedfs/templates/sftp/sftp-deployment.yaml | 4 ++-- k8s/charts/seaweedfs/templates/volume/volume-statefulset.yaml | 4 ++-- k8s/charts/seaweedfs/templates/worker/worker-deployment.yaml | 4 ++-- k8s/charts/seaweedfs/values.yaml | 4 ++-- 10 files changed, 20 insertions(+), 20 deletions(-) diff --git a/k8s/charts/seaweedfs/templates/admin/admin-statefulset.yaml b/k8s/charts/seaweedfs/templates/admin/admin-statefulset.yaml index 302813046..9bcb08e75 100644 --- a/k8s/charts/seaweedfs/templates/admin/admin-statefulset.yaml +++ b/k8s/charts/seaweedfs/templates/admin/admin-statefulset.yaml @@ -119,7 +119,7 @@ spec: {{- range $key, $value := .Values.admin.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} @@ -130,7 +130,7 @@ spec: {{- range $key, $value := .Values.global.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} diff --git a/k8s/charts/seaweedfs/templates/all-in-one/all-in-one-deployment.yaml b/k8s/charts/seaweedfs/templates/all-in-one/all-in-one-deployment.yaml index f6237bb7e..fc4ac13a3 100644 --- a/k8s/charts/seaweedfs/templates/all-in-one/all-in-one-deployment.yaml +++ b/k8s/charts/seaweedfs/templates/all-in-one/all-in-one-deployment.yaml @@ -104,7 +104,7 @@ spec: {{- if and (ne $key $clusterMasterKey) (ne $key $clusterFilerKey) }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 }} @@ -117,7 +117,7 @@ spec: {{- if and (ne $key $clusterMasterKey) (ne $key $clusterFilerKey) }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 }} diff --git a/k8s/charts/seaweedfs/templates/cosi/cosi-deployment.yaml b/k8s/charts/seaweedfs/templates/cosi/cosi-deployment.yaml index 813af850d..a040541d7 100644 --- a/k8s/charts/seaweedfs/templates/cosi/cosi-deployment.yaml +++ b/k8s/charts/seaweedfs/templates/cosi/cosi-deployment.yaml @@ -100,7 +100,7 @@ spec: {{- range $key, $value := .Values.cosi.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} @@ -111,7 +111,7 @@ spec: {{- range $key, $value := .Values.global.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} diff --git a/k8s/charts/seaweedfs/templates/filer/filer-statefulset.yaml b/k8s/charts/seaweedfs/templates/filer/filer-statefulset.yaml index b7df27158..d6605ab01 100644 --- a/k8s/charts/seaweedfs/templates/filer/filer-statefulset.yaml +++ b/k8s/charts/seaweedfs/templates/filer/filer-statefulset.yaml @@ -118,7 +118,7 @@ spec: {{- range $key, $value := .Values.filer.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} @@ -129,7 +129,7 @@ spec: {{- range $key, $value := .Values.global.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} diff --git a/k8s/charts/seaweedfs/templates/master/master-statefulset.yaml b/k8s/charts/seaweedfs/templates/master/master-statefulset.yaml index 50e0e97d4..dca3000c6 100644 --- a/k8s/charts/seaweedfs/templates/master/master-statefulset.yaml +++ b/k8s/charts/seaweedfs/templates/master/master-statefulset.yaml @@ -102,7 +102,7 @@ spec: {{- range $key, $value := .Values.master.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} @@ -113,7 +113,7 @@ spec: {{- range $key, $value := .Values.global.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} diff --git a/k8s/charts/seaweedfs/templates/s3/s3-deployment.yaml b/k8s/charts/seaweedfs/templates/s3/s3-deployment.yaml index 29dd2d434..ad8b9530e 100644 --- a/k8s/charts/seaweedfs/templates/s3/s3-deployment.yaml +++ b/k8s/charts/seaweedfs/templates/s3/s3-deployment.yaml @@ -94,7 +94,7 @@ spec: {{- range $key, $value := .Values.s3.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} @@ -105,7 +105,7 @@ spec: {{- range $key, $value := .Values.global.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} diff --git a/k8s/charts/seaweedfs/templates/sftp/sftp-deployment.yaml b/k8s/charts/seaweedfs/templates/sftp/sftp-deployment.yaml index c0bcb2c4a..6197be6b3 100644 --- a/k8s/charts/seaweedfs/templates/sftp/sftp-deployment.yaml +++ b/k8s/charts/seaweedfs/templates/sftp/sftp-deployment.yaml @@ -94,7 +94,7 @@ spec: {{- range $key, $value := .Values.sftp.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} @@ -105,7 +105,7 @@ spec: {{- range $key, $value := .Values.global.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} diff --git a/k8s/charts/seaweedfs/templates/volume/volume-statefulset.yaml b/k8s/charts/seaweedfs/templates/volume/volume-statefulset.yaml index 045b95c2e..1ac5c933a 100644 --- a/k8s/charts/seaweedfs/templates/volume/volume-statefulset.yaml +++ b/k8s/charts/seaweedfs/templates/volume/volume-statefulset.yaml @@ -122,7 +122,7 @@ spec: {{- range $key, $value := $volume.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} @@ -133,7 +133,7 @@ spec: {{- range $key, $value := $.Values.global.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} diff --git a/k8s/charts/seaweedfs/templates/worker/worker-deployment.yaml b/k8s/charts/seaweedfs/templates/worker/worker-deployment.yaml index 7a6e05243..773dc7fa3 100644 --- a/k8s/charts/seaweedfs/templates/worker/worker-deployment.yaml +++ b/k8s/charts/seaweedfs/templates/worker/worker-deployment.yaml @@ -97,7 +97,7 @@ spec: {{- range $key, $value := .Values.worker.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} @@ -108,7 +108,7 @@ spec: {{- range $key, $value := .Values.global.extraEnvironmentVars }} - name: {{ $key }} {{- if kindIs "string" $value }} - value: {{ $value | quote }} + value: {{ tpl $value $ | quote }} {{- else }} valueFrom: {{ toYaml $value | nindent 16 | trim }} diff --git a/k8s/charts/seaweedfs/values.yaml b/k8s/charts/seaweedfs/values.yaml index 7bad4e2d9..17b3f44d1 100644 --- a/k8s/charts/seaweedfs/values.yaml +++ b/k8s/charts/seaweedfs/values.yaml @@ -39,8 +39,8 @@ global: replicationPlacement: "001" extraEnvironmentVars: WEED_CLUSTER_DEFAULT: "sw" - WEED_CLUSTER_SW_MASTER: "seaweedfs-master.seaweedfs:9333" - WEED_CLUSTER_SW_FILER: "seaweedfs-filer-client.seaweedfs:8888" + WEED_CLUSTER_SW_MASTER: "{{ include \"seaweedfs.cluster.masterAddress\" . }}" + WEED_CLUSTER_SW_FILER: "{{ include \"seaweedfs.cluster.filerAddress\" . }}" # WEED_JWT_SIGNING_KEY: # secretKeyRef: # name: seaweedfs-signing-key