pyttel commented on code in PR #10:
URL: https://github.com/apache/ozone-helm-charts/pull/10#discussion_r1832177011


##########
charts/ozone/templates/scm/scm-statefulset.yaml:
##########
@@ -61,6 +62,24 @@ spec:
               mountPath: {{ .Values.configuration.dir }}
             - name: {{ .Release.Name }}-scm
               mountPath: {{ .Values.scm.persistence.path }}
+        {{- if gt (int .Values.scm.replicas) 1 }}
+        - name: bootstrap
+          image: "{{ .Values.image.repository }}:{{ .Values.image.tag | 
default .Chart.AppVersion }}"
+          args: ["ozone", "scm", "--bootstrap"]

Review Comment:
   > It appears from the documentation that init on pod-1 needs to complete 
before bootstrap on pod-x. This would perhaps require changing 
podManagementPolicy: Parallel to OrderedReady.
   
   I used this because of the ratis ring. The problem with the other 
configuration is that the hosts of unstated nodes from the headless service 
cannot be resolved. So the first SCM node cannot start because it depends on 
the resolution of the other ones, which again cannot start because they start 
only if first node has finished successfully



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to