[pulsar-helm-chart] branch master updated: Fix bookkeeper metadata init when specifying metadataPrefix (#316)

[mmarshall]

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

mmarshall 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 9324a9a  Fix bookkeeper metadata init when specifying metadataPrefix 
(#316)
9324a9a is described below

commit 9324a9a27065204182d04c1f8dc64d800fbb39ee
Author: Michael Marshall <mmarsh...@apache.org>
AuthorDate: Thu Oct 20 13:24:20 2022 -0700

    Fix bookkeeper metadata init when specifying metadataPrefix (#316)
    
    Fixes #309
    
    ### Motivation
    
    Fix the metadataPrefix initialization.
    
    ### Modifications
    
    * Fix the script by adding `&& echo`
    
    ### Verifying this change
    
    I manually verified that this change works and correctly puts the metadata 
in the prefixed location.
---
 charts/pulsar/templates/bookkeeper-cluster-initialize.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/charts/pulsar/templates/bookkeeper-cluster-initialize.yaml 
b/charts/pulsar/templates/bookkeeper-cluster-initialize.yaml
index 928ddcf..c66f6f2 100644
--- a/charts/pulsar/templates/bookkeeper-cluster-initialize.yaml
+++ b/charts/pulsar/templates/bookkeeper-cluster-initialize.yaml
@@ -74,7 +74,7 @@ spec:
                 echo "bookkeeper cluster already initialized";
             else
                 {{- if not (eq .Values.metadataPrefix "") }}
-                bin/bookkeeper org.apache.zookeeper.ZooKeeperMain -server {{ 
template "pulsar.fullname" . }}-{{ .Values.zookeeper.component }} create {{ 
.Values.metadataPrefix }} 'created for pulsar cluster "{{ template 
"pulsar.cluster.name" . }}"' &&
+                bin/bookkeeper org.apache.zookeeper.ZooKeeperMain -server {{ 
template "pulsar.fullname" . }}-{{ .Values.zookeeper.component }} create {{ 
.Values.metadataPrefix }} && echo 'created for pulsar cluster "{{ template 
"pulsar.cluster.name" . }}"' &&
                 {{- end }}
                 bin/bookkeeper shell initnewcluster;
             fi