smbecker commented on PR #674: URL: https://github.com/apache/pulsar-helm-chart/pull/674#issuecomment-4299106929
Thanks for the thorough review. The latest commit should cover all of the items. One thing to call out, I followed the broker's pattern (keep non-TLS webServicePort open for metrics/prometheus, clear only the Pulsar wire-protocol listener) rather than clearing both because the podMonitor/prometheus.io/port scrape is still HTTP. -- 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]
