Comment out new option in helm values, because it breaks the old vers…

…ions of keda (the cmd arg does not exist for them), this way the default value is still correctly applied for new keda version because they have the fallback implemented also in the golang code (kedacore#659)

Signed-off-by: Jirka Kremser <[email protected]>

keda/templates/manager/deployment.yaml

@@ -103,9 +103,11 @@ spec:
           {{- if .Values.profiling.operator.enabled }}
           - "--profiling-bind-address=:{{ .Values.profiling.operator.port }}"
           {{- end }}
-          {{- range .Values.certificates.operator.caDirs }}
+          {{- with (.Values.certificates.operator).caDirs }}
+          {{- range . }}
           - "--ca-dir={{ . }}"
           {{- end }}
+          {{- end }}
           {{- range $key, $value := .Values.extraArgs.keda }}
           - "--{{ $key }}={{ $value }}"
           {{- end }}

keda/values.yaml

@@ -817,8 +817,8 @@ certificates:
       group: cert-manager.io
   operator:
     # -- Location(s) of CA files for authentication of external TLS connections such as TLS-enabled metrics sources
-    caDirs:
-    - /custom/ca
+    # caDirs:
+    # - /custom/ca
 
 permissions:
   metricServer: