lhotari opened a new pull request, #553:
URL: https://github.com/apache/pulsar-helm-chart/pull/553

   Fixes #551
   
   ### Motivation
   
   There's an issue in the current Oxia configuration in Apache Pulsar Helm 
chart described in #551
   
   ### Modifications
   
   The config.yaml was invalid. In the original Oxia Helm chart, all pods of 
the statefulset are listed:
   
https://github.com/streamnative/oxia/blob/79fea6ac5a5ba61572505149a7024a46bb936ded/deploy/charts/oxia-cluster/templates/coordinator-configmap.yaml#L29-L32
   A subtle mistake was made when this template was added to the Apache Pulsar 
Helm chart.
   This PR adds similar logic to the configuration. The template logic contains 
variable names that make it easy to follow the logic.
   
   ### Verifying this change
   
   - [ ] Make sure that the change passes the CI checks.
   


-- 
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]

Reply via email to