Mortom123 opened a new pull request, #448: URL: https://github.com/apache/pulsar-helm-chart/pull/448
See https://github.com/apache/pulsar-helm-chart/pull/438 / https://github.com/apache/pulsar-helm-chart/pull/438#issuecomment-1903889439 3.PR Also the Pulsar-Manager Admin interface is exposed on a different, non outward facing, service because of [given concerns](https://github.com/apache/pulsar-helm-chart/pull/440#issuecomment-1909602387). ### Motivation Right now you need to initialize the pulsar-manager manually, this can be done as an init job. ### Modifications 1. Change Admin Port exposure of Pulsar Manager to second internal service 2. Provide means to securely generate a password / use an existing one 3. Provide a job that runs on init and conditions the Pulsar Manager ### Verifying this change - [] Make sure that the change passes the CI checks. [As of right now still running](https://github.com/Mortom123/pulsar-helm-chart/actions/runs/7687391348) Some things I also noted: 1. I tried all combinations of BROKER_URL and BOOKIE_URL such that BOOKIE_URL references the Bookie service and one of its ports (as given in the values). However, none of them worked. I suspect there's a naming error on side of the Pulsar Manager 2. There are some errors when starting the manager. Maybe part of this deployment is incorrect. ``` /pulsar-manager/startup.sh: 21: initdb: not found /pulsar-manager/startup.sh: 22: pg_ctl: not found ``` -- 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]
