apache / pulsar-helm-chart

Official Apache Pulsar Helm Chart
https://pulsar.apache.org/
Apache License 2.0
212 stars 224 forks source link

Add support for admin port on ZooKeeper #550

Closed liamgib closed 1 week ago

liamgib commented 1 week ago

Fixes #549

Motivation

We need to be able to access the ZooKeeper admin endpoint to perform automated snapshots to an external system.

Modifications

I have added the admin port to the list of Zookeeper ports, and made the relevant changes on the zookeeper-service.yaml & zookeeper-statefulset.yaml to use the admin port.

Verifying this change

liamgib commented 1 week ago

Good point @lhotari, there is some security implications. Especially for existing clusters, where they might not be expecting the admin port being exposed.

I'll update the PR to make the field optional, and removed by default.