lhotari opened a new pull request, #684: URL: https://github.com/apache/pulsar-helm-chart/pull/684
Adds .ci/templates-all-values.yaml (every optional feature on, with multi-element lists and maps for every list/map-shaped value) and .ci/templates-all-values-patch1.yaml (overlay that flips mutually-exclusive choices: zookeeper -> oxia, JWT symmetric -> OpenID, selfsigning -> ca issuer, multiVolumes -> single common bookkeeper volume, hard -> soft anti-affinity, etc.). A new step in the ct-lint job renders both combinations with `helm template` and validates them with `kubeconform -strict` against k8s 1.34.0. This catches indent / `range` / `toYaml | nindent` bugs that no individual scenario in .ci/clusters/ can surface on its own, since each existing scenario only enables one or two features at a time. Also fixes two real bugs uncovered by writing the test: oxia-coordinator-serviceaccount.yaml and oxia-server-serviceaccount.yaml treated `images.imagePullSecrets` as an object with `.secretName`, but per the schema in values.yaml and the `pulsar.imagePullSecrets` helper, it is a list of secret-name strings. Both files now use the shared helper and template-error no longer when the field is set. -- 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]
