Anubhav Jindal has posted comments on this change. ( http://gerrit.cloudera.org:8080/24344 )
Change subject: IMPALA-14978: Add runnable Helm values examples ...................................................................... Patch Set 23: > Patch Set 22: > > Question on values-ldap-example.yaml: Is this intended to be a complete > runnable Impala LDAP example in this CL, or scaffolding for follow-up changes? > > Right now it appears to include mostly OpenLDAP-style keys and does not set > auth.ldap.* values used by the Impala chart, and it also uses > persistence.storageClass (the chart uses persistence.storageClassName). > > If follow-up work is planned, should we either rename/document this file as > partial, or make it self-contained here so users can run it without extra > --set auth.ldap... overrides? Thanks Gokul. `values-ldap-example.yaml` is intentionally for the OpenLDAP chart (`openldap/openldap`), not the Impala chart. That’s why it uses OpenLDAP-style keys and `persistence.storageClass`. For Impala LDAP auth, I added `values-impala-ldap-example.yaml` with `auth.ldap.*` preconfigured, so users can deploy Impala LDAP without extra `--set auth.ldap...` overrides. I also updated the LDAP docs in IMPALA-14982 to make this two-file flow explicit. -- To view, visit http://gerrit.cloudera.org:8080/24344 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib5c6ab7386bdd791a057b6402e15a82d97c1c381 Gerrit-Change-Number: 24344 Gerrit-PatchSet: 23 Gerrit-Owner: Anubhav Jindal <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Anubhav Jindal <[email protected]> Gerrit-Reviewer: Gokul Kolady <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Comment-Date: Fri, 12 Jun 2026 05:28:04 +0000 Gerrit-HasComments: No
