Jason Fehr has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24344 )
Change subject: IMPALA-14978: Add runnable Helm values examples ...................................................................... IMPALA-14978: Add runnable Helm values examples Add runnable values files to document common starting configurations for Kubernetes deployments. Include a minimal Impala example (values-example.yaml), an OpenLDAP chart example (values-ldap-example.yaml), and a self-contained Impala LDAP auth example (values-impala-ldap-example.yaml). Clarify that values-ldap-example.yaml is for openldap/openldap, not the Impala chart. Align shared warehouse storage access mode with multi-node safe defaults and add ASF license headers for new example files. Testing: - helm lint helm/impala - helm template impala14978-example helm/impala -f helm/impala/values-example.yaml - helm repo add openldap https://jp-gouin.github.io/helm-openldap/ - helm repo update - helm template impala14978-openldap openldap/openldap -f helm/impala/values-ldap-example.yaml - helm template impala14978-impala-ldap helm/impala -f helm/impala/values-impala-ldap-example.yaml - kubectl config current-context (k3d-impala-live) - kubectl create namespace impala-14978-live - helm upgrade --install impala-14978-live helm/impala -n impala-14978-live \ -f helm/impala/values-example.yaml --set persistence.accessModes[0]=ReadWriteOnce - kubectl rollout status deployment/impala-14978-live-impala-{statestored,catalogd,impalad,hms} \ -n impala-14978-live - kubectl get pods -n impala-14978-live (all Running/Ready) - kubectl create namespace impala-14978-ldap-live - helm upgrade --install impala-14978-ldap-live helm/impala -n impala-14978-ldap-live \ -f helm/impala/values-impala-ldap-example.yaml --set persistence.accessModes[0]=ReadWriteOnce - kubectl rollout status deployment/impala-14978-ldap-live-impala-{statestored,catalogd,impalad,hms} \ -n impala-14978-ldap-live - kubectl get deployment impala-14978-ldap-live-impala-impalad -n impala-14978-ldap-live \ -o jsonpath='{.spec.template.spec.containers[0].args}' (contains -enable_ldap_auth) Change-Id: Ib5c6ab7386bdd791a057b6402e15a82d97c1c381 Assisted-by: GPT-5.3 (Cursor) Reviewed-on: http://gerrit.cloudera.org:8080/24344 Reviewed-by: Gokul Kolady <[email protected]> Reviewed-by: Jason Fehr <[email protected]> Tested-by: Jason Fehr <[email protected]> --- A helm/impala/values-example.yaml A helm/impala/values-impala-ldap-example.yaml A helm/impala/values-ldap-example.yaml 3 files changed, 174 insertions(+), 0 deletions(-) Approvals: Gokul Kolady: Looks good to me, but someone else must approve Jason Fehr: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: Ib5c6ab7386bdd791a057b6402e15a82d97c1c381 Gerrit-Change-Number: 24344 Gerrit-PatchSet: 29 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]>
