From 48501ebe84b5b339189ee145c6cf5c291aff6063 Mon Sep 17 00:00:00 2001 From: Michael Marshall Date: Mon, 17 Oct 2022 15:59:05 -0700 Subject: [PATCH] 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 }}