Closed lhotari closed 6 days ago
Fixes #551
There's an issue in the current Oxia configuration in Apache Pulsar Helm chart described in #551
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.
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