Hello Gokul Kolady, Abhishek Rawat, Jason Fehr, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24335
to look at the new patch set (#8).
Change subject: IMPALA-14975: Add HMS config templating to Helm
......................................................................
IMPALA-14975: Add HMS config templating to Helm
Add chart-managed hive-site.xml and a release-scoped ConfigMap template
consumed by Impala and HMS pods.
Template the metastore URI to the release-local HMS service name so chart
deployments avoid static host assumptions.
Add ASF license headers for the newly added chart template file.
Testing:
- helm lint helm/impala
- helm template impala14975-check helm/impala
- kubectl config current-context (k3d-impala-live)
- kubectl create namespace impala-14975-live
- helm upgrade --install impala-14975-live helm/impala -n impala-14975-live
--set persistence.accessModes[0]=ReadWriteOnce
- kubectl rollout status
deployment/impala-14975-live-impala-{statestored,catalogd,impalad,hms} -n
impala-14975-live
- kubectl get pods -n impala-14975-live (all Running/Ready)
- kubectl get configmap impala-14975-live-impala-conf -n impala-14975-live -o
jsonpath='{.data.hive-site\.xml}' (contains
thrift://impala-14975-live-impala-hms:9083)
- kubectl get deployment
impala-14975-live-impala-{statestored,catalogd,impalad,hms} -n
impala-14975-live -o yaml (all mount hive-site.xml from conf configmap)
Implemented and validated manually; Cursor-assisted.
Change-Id: I04c67c1b8bd7c75f5cf9fb7e86084fcb7dd8db83
Co-authored-by: Cursor <[email protected]>
---
A helm/impala/files/hive-site.xml
A helm/impala/templates/configmap.yaml
2 files changed, 97 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/35/24335/8
--
To view, visit http://gerrit.cloudera.org:8080/24335
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I04c67c1b8bd7c75f5cf9fb7e86084fcb7dd8db83
Gerrit-Change-Number: 24335
Gerrit-PatchSet: 8
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]>