Do not attempt to check TLS state for dry-runs

redpanda-data · Jun 1, 2023 · cb72a73 · cb72a73
1 parent d9ac082
commit cb72a73
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/charts/redpanda/Chart.yaml b/charts/redpanda/Chart.yaml
@@ -23,7 +23,7 @@ type: application
 # The chart version and the app version are not the same and will not track
 # together. The chart version is a semver representation of changes to this
 # chart.
-version: 4.0.29
+version: 4.0.30
 
 # The app version is the default version of Redpanda to install.
 appVersion: v23.1.10

diff --git a/charts/redpanda/templates/configmap.yaml b/charts/redpanda/templates/configmap.yaml
@@ -32,7 +32,8 @@ limitations under the License.
 {{- $cm := lookup "v1" "ConfigMap" .Release.Namespace (include "redpanda.fullname" .) -}}
 {{- $redpandaYAML := dig "data" "redpanda.yaml" "" $cm | fromYaml -}}
 {{- $currentRPCTLS := dig "redpanda" "rpc_server_tls" "enabled" false $redpandaYAML -}}
-{{- if .Release.IsUpgrade -}}
+{{- /* Lookup will return an empty map when running `helm template` or when `--dry-run` is passed. */ -}}
+{{- if (and .Release.IsUpgrade $cm) -}}
   {{- if ne $currentRPCTLS $wantedRPCTLS -}}
     {{- if eq (get .Values "force" | default false) false -}}
       {{- fail (join "\n" (list