Skip to content

Commit 14026b4

Browse files
committed
Fix legacy tls detection
When cluster is using legacy tls the upgrade fails when cluster name is also explicitely overriden because helm wasn't using the full tls lookup path.
1 parent c5dd7a5 commit 14026b4

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

charts/couchbase-operator/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
apiVersion: v2
22
name: couchbase-operator
33
description: A Helm chart to deploy the Couchbase Autonomous Operator for easily deploying, managing, and maintaining Couchbase Clusters. Couchbase Server is a NoSQL document database with a distributed architecture for performance, scalability, and availability. It enables developers to build applications easier and faster by leveraging the power of SQL with the flexibility of JSON.
4-
version: 2.32.0
4+
version: 2.32.1
55
appVersion: 2.3.2
66
type: application
77
keywords:

charts/couchbase-operator/templates/_helpers.tpl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -294,7 +294,7 @@ Determine if tls legacy mode is enabled. Legacy TLS involves use of static secr
294294
{{- $clusterName := (include "couchbase-cluster.clustername" .) -}}
295295
{{- $clusterSpec := (lookup "couchbase.com/v2" "CouchbaseCluster" .Release.Namespace $clusterName) -}}
296296
{{- if $clusterSpec -}}
297-
{{- $clusterTLS := $clusterSpec.spec.networking -}}
297+
{{- $clusterTLS := $clusterSpec.spec.networking.tls -}}
298298
{{- if $clusterTLS -}}
299299
{{- if $clusterTLS.static -}}
300300
{{/* legacy format is in use for cluster */}}

0 commit comments

Comments
 (0)