[pulsar-helm-chart] branch master updated: Allow bk cluster init to restart on failure (#303)

[mm...@apache.org]

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 48501eb  Allow bk cluster init to restart on failure (#303)
48501eb is described below

commit 48501ebe84b5b339189ee145c6cf5c291aff6063
Author: Michael Marshall <mm...@apache.org>
AuthorDate: Mon Oct 17 15:59:05 2022 -0700

    Allow bk cluster init to restart on failure (#303)
    
    ### Motivation
    
    This is essentially the same as https://github.com/apache/pulsar-helm-chart/pull/176. Without this change, an init pod can fail and be in `Error` state even though the second pod succeeded. This will prevent misleading errors.
    
    ### Modifications
    
    * Replace `Never` with `OnFailure`
    
    ### Verifying this change
    
    This is a trivial change.
---
 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 cedb648..5e54b18 100644
--- a/charts/pulsar/templates/bookkeeper-cluster-initialize.yaml
+++ b/charts/pulsar/templates/bookkeeper-cluster-initialize.yaml
@@ -88,6 +88,6 @@ spec:
         {{- include "pulsar.toolset.certs.volumeMounts" . | nindent 8 }}
       volumes:
       {{- include "pulsar.toolset.certs.volumes" . | nindent 6 }}
-      restartPolicy: Never
+      restartPolicy: OnFailure
 {{- end }}
 {{- end }}