Hello Gokul Kolady, Jason Fehr, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24342
to look at the new patch set (#8).
Change subject: IMPALA-14976: Add optional Kudu Helm resources
......................................................................
IMPALA-14976: Add optional Kudu Helm resources
Add optional Kudu master/tserver StatefulSets, services, and PVC templates
behind chart flags so Kudu-backed Impala deployments can be enabled when
needed.
Add chart validation that enforces a single-replica Kudu baseline for
master and tserver deployments to avoid unsupported multi-replica behavior
with the current storage and service model.
Add ASF license headers to newly added Kudu chart templates.
Testing:
- helm lint helm/impala
- helm template impala14976-check helm/impala
- helm template impala14976-kudu-on helm/impala --set kudu.enabled=true
- helm template impala14976-kudu-master-bad helm/impala --set kudu.enabled=true
--set kudu.master.replicas=2 (fails as expected)
- helm template impala14976-kudu-tserver-bad helm/impala --set
kudu.enabled=true --set kudu.tserver.replicas=2 (fails as expected)
- kubectl config current-context (k3d-impala-live)
- helm upgrade --install impala-kudu-live helm/impala -n impala-kudu-14976-live
--set kudu.enabled=true --set persistence.accessModes[0]=ReadWriteOnce with
local clock overrides for Kudu extraArgs
- kubectl rollout status
deployment/impala-kudu-live-impala-{statestored,catalogd,impalad,hms} -n
impala-kudu-14976-live
- kubectl rollout status
statefulset/impala-kudu-live-impala-kudu-{master,tserver} -n
impala-kudu-14976-live
- kubectl get pods -n impala-kudu-14976-live (all Running/Ready)
- kubectl get deployment impala-kudu-live-impala-impalad -n
impala-kudu-14976-live -o jsonpath='{.spec.template.spec.containers[0].args}'
(contains -kudu_master_hosts=impala-kudu-live-impala-kudu-master:7051)
- kubectl exec impala-kudu-live-impala-kudu-master-0 -n impala-kudu-14976-live
-- kudu cluster ksck impala-kudu-live-impala-kudu-master:7051 (OK)
Implemented and validated manually; Cursor-assisted.
Change-Id: I50671c46bbc30beb9d643880910adbd3f77407b6
Co-authored-by: Cursor <[email protected]>
---
A helm/impala/templates/kudu-master-deployment.yaml
A helm/impala/templates/kudu-master-service.yaml
A helm/impala/templates/kudu-pvc.yaml
A helm/impala/templates/kudu-tserver-deployment.yaml
A helm/impala/templates/kudu-tserver-service.yaml
A helm/impala/templates/kudu-validation.yaml
M helm/impala/values.yaml
7 files changed, 362 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/42/24342/8
--
To view, visit http://gerrit.cloudera.org:8080/24342
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I50671c46bbc30beb9d643880910adbd3f77407b6
Gerrit-Change-Number: 24342
Gerrit-PatchSet: 8
Gerrit-Owner: Anubhav Jindal <[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]>
