pyttel commented on code in PR #10:
URL: https://github.com/apache/ozone-helm-charts/pull/10#discussion_r1832161453
##########
charts/ozone/templates/om/om-service-headless.yaml:
##########
@@ -28,6 +28,10 @@ spec:
ports:
- name: ui
port: {{ .Values.om.service.port }}
+ {{- if gt (int .Values.om.replicas) 1 }}
+ - name: ratis
+ port: 9872
+ {{- end }}
Review Comment:
> Is this port being exposed for one Ozone Manager(OM) to communicate with
another OM when they form a Ratis ring?
yes this was my reason
> My understanding is that the current selector will match this service with
all OM pods. Consequently, the messages sent to this service will be forwarded
to a random OM pod instead of a specific OM pod.
ok if this is the case maybe we can remove this export. I found and used the
following Ticket for the port configuration:
https://issues.apache.org/jira/browse/HDDS-4677. I'm not really familiar with
the architecture. If we can remove it I will do so :)
--
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]
