Tejaskriya commented on PR #10: URL: https://github.com/apache/ozone-helm-charts/pull/10#issuecomment-3266348311
@pyttel I'm quite new to Kubernetes so do excuse me if I've gotten anything wrong. With the implementation you have proposed, if a user had to remove some OMs, they would reduce the number of replicas. Using the helm hooks, we would decommission the required OMs. But this logic always decommissions the highest numbered OM right? i.e., `om-<highest number>`. This blocks the user from being able to decommission any OM of their choice. This would be restrictive, but for an initial implementation for enabling HA, I think this logic works fine. We might need to revisit it later on. Do you plan to change the hardcoding of the port numbers in this PR? I also agree with @ptlrs 's suggestion to avoid hardcoding it. -- 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]
