apache / pulsar-helm-chart

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

Broker environnement variable cannot be set #539

Closed lenglet-k closed 1 month ago

lenglet-k commented 1 month ago

Describe the bug If we want add environment variable with extreEnvs object in broker is not possible because env: is declared twice in yaml manifest.

Here: https://github.com/apache/pulsar-helm-chart/blob/master/charts/pulsar/templates/broker-statefulset.yaml#L266 and Here: https://github.com/apache/pulsar-helm-chart/blob/master/charts/pulsar/templates/broker-statefulset.yaml#L298

To Reproduce Steps to reproduce the behavior:

  1. Set environment variable in broker.extreEnvs
  2. Deploy helm release
  3. See your pod deployed by broker statefulset without your environment variable

Expected behavior Set environment variable for broker