pyttel commented on code in PR #10:
URL: https://github.com/apache/ozone-helm-charts/pull/10#discussion_r1834872944
##########
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
However, this is only relevant for the bootstrap process. So, you might be
right. We probably only need the bootstrap once in persistent mode. Maybe
someone from the Ozone contributors can provide a definitive answer to this
question.
--
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]
