This is an automated email from the ASF dual-hosted git repository.

lhotari pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar-helm-chart.git


The following commit(s) were added to refs/heads/master by this push:
     new dd879cf  Revisit configurationMetadataStoreUrl and 
bookkeeperMetadataServiceUri settings (#652)
dd879cf is described below

commit dd879cf4576e6086480162d224803907fcf1944c
Author: Lari Hotari <[email protected]>
AuthorDate: Mon Feb 23 23:13:23 2026 +0200

    Revisit configurationMetadataStoreUrl and bookkeeperMetadataServiceUri 
settings (#652)
    
    - Set configurationMetadataStoreUrl only if it's different than 
metadataStoreUrl
    - Don't set bookkeeperMetadataServiceUri if 
usePulsarMetadataClientDriver=true
---
 charts/pulsar/templates/broker-configmap.yaml | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/charts/pulsar/templates/broker-configmap.yaml 
b/charts/pulsar/templates/broker-configmap.yaml
index 29f8970..2fde143 100644
--- a/charts/pulsar/templates/broker-configmap.yaml
+++ b/charts/pulsar/templates/broker-configmap.yaml
@@ -30,22 +30,15 @@ data:
   # Metadata settings
   {{- if .Values.components.zookeeper }}
   metadataStoreUrl: "zk:{{ template "pulsar.zookeeper.connect" . }}{{ 
.Values.metadataPrefix }}"
-  {{- $configMetadataStoreUrl := "" }}
   {{- if .Values.pulsar_metadata.configurationStore }}
-  {{- $configMetadataStoreUrl = printf "zk:%s%s" (include 
"pulsar.configurationStore.connect" .) 
.Values.pulsar_metadata.configurationStoreMetadataPrefix }}
-  {{- else }}
-  {{- $configMetadataStoreUrl = printf "zk:%s%s" (include 
"pulsar.zookeeper.connect" .) .Values.metadataPrefix }}
+  configurationMetadataStoreUrl: "zk:{{ template 
"pulsar.configurationStore.connect" . }}{{ 
.Values.pulsar_metadata.configurationStoreMetadataPrefix }}"
   {{- end }}
-  configurationMetadataStoreUrl: "{{ $configMetadataStoreUrl }}"
-  {{- if .Values.pulsar_metadata.bookkeeper.usePulsarMetadataClientDriver }}
-  bookkeeperMetadataServiceUri: "metadata-store:{{ $configMetadataStoreUrl 
}}/ledgers"
-  {{- else }}
+  {{- if not .Values.pulsar_metadata.bookkeeper.usePulsarMetadataClientDriver 
}}
   bookkeeperMetadataServiceUri: "zk+hierarchical://{{ template 
"pulsar.zookeeper.connect" . }}{{ .Values.metadataPrefix }}/ledgers"
   {{- end }}
   {{- end }}
   {{- if .Values.components.oxia }}
   metadataStoreUrl: "{{ template "pulsar.oxia.metadata.url.broker" . }}"
-  configurationMetadataStoreUrl: "{{ template 
"pulsar.oxia.metadata.url.broker" . }}"
   bookkeeperMetadataServiceUri: "{{ template 
"pulsar.oxia.metadata.url.bookkeeper" . }}"
   {{- end }}
 

Reply via email to