fix: watchNamespace value does not take effect in chart (#990)

Signed-off-by: drivebyer <[email protected]>
OT-CONTAINER-KIT · Jun 15, 2024 · 052d2e5 · 052d2e5
1 parent 4afc8c2
commit 052d2e5
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 7 deletions.
diff --git a/charts/redis-operator/Chart.yaml b/charts/redis-operator/Chart.yaml
@@ -1,6 +1,6 @@
 ---
 apiVersion: v2
-version: 0.16.2
+version: 0.16.3
 appVersion: "0.17.0"
 description: Provides easy redis setup definitions for Kubernetes services, and deployment.
 engine: gotpl

diff --git a/charts/redis-operator/templates/operator-deployment.yaml b/charts/redis-operator/templates/operator-deployment.yaml
@@ -48,10 +48,10 @@ spec:
       {{- end }}
         env:
         - name: ENABLE_WEBHOOKS
-          value: "{{ .Values.redisOperator.webhook | toString }}"
-        {{- if .WATCH_NAMESPACE }}
+          value: {{ .Values.redisOperator.webhook | quote }}
+        {{- if .Values.redisOperator.watchNamespace }}
         - name: WATCH_NAMESPACE
-          value: {{ .WATCH_NAMESPACE }}
+          value: {{ .Values.redisOperator.watchNamespace | quote }}
         {{- end }}
         {{- range $env := .Values.redisOperator.env }}
         - name: {{ $env.name }}

diff --git a/charts/redis-operator/values.yaml b/charts/redis-operator/values.yaml
@@ -13,9 +13,8 @@ redisOperator:
 
   # Additional arguments for redis-operator container
   extraArgs: []
-
-  #  - -zap-log-level=error
-  watch_namespace: ""
+  # When not specified, the operator will watch all namespaces. It can be set to a specific namespace or multiple namespaces separated by commas.
+  watchNamespace: ""
   env: []
   webhook: false