redpanda-data / helm-charts

Redpanda Helm Chart
http://redpanda.com
Apache License 2.0
77 stars 96 forks source link

Users specified in `auth.sasl.secretRef` get overridden as of redpanda-5.9.3 #1566

Closed chrisseto closed 3 weeks ago

chrisseto commented 1 month ago

What happened?

Only the kubernetes-controller user is added as a superuser.

What did you expect to happen?

All users within auth.sasl.secretRef to be created as superusers.

How can we reproduce it (as minimally and precisely as possible)?. Please include values file.

auth:
  sasl:
    enabled: true
    secretRef: "some-secret"

Anything else we need to know?

The offending change is (very subtly) here: https://github.com/redpanda-data/helm-charts/commit/081c08b6b83ba196994ec3312a7c6011e4ef0a22#diff-84c6555620e4e5f79262384a9fa3e8f4876b36bb3a64748cbd8fbdcb66e8c1b9R247

Which are the affected charts?

Redpanda

Chart Version(s)

redpanda-5.9.3

Cloud provider

N/A

JIRA Link: K8S-389

chrisseto commented 3 weeks ago

Fixed in https://github.com/redpanda-data/redpanda-operator/pull/265