Gokul Kolady has posted comments on this change. ( http://gerrit.cloudera.org:8080/24344 )
Change subject: IMPALA-14978: Add runnable Helm values examples ...................................................................... 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? -- 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: 22 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: Tue, 09 Jun 2026 23:19:40 +0000 Gerrit-HasComments: No
