Hello Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24365
to look at the new patch set (#13).
Change subject: IMPALA-14983: Add typed operator config keys
......................................................................
IMPALA-14983: Add typed operator config keys
Add typed configuration fields under ImpalaCluster spec.config for
impalad, catalogd, statestored, and HMS daemon flags plus impalad query
defaults.
Map the typed fields into Helm values in the operator reconciler while
keeping spec.set as a backward-compatible advanced override path.
Add focused operator unit tests covering typed set-arg rendering,
escaping behavior, and LDAP disable/reconciliation edge cases.
Update CRD schema, sample ImpalaCluster, and Kubernetes deployment guide
with typed-key examples and escaping guidance.
Testing:
- python3 -m py_compile operator/impala-operator/main.py
- python3 operator/impala-operator/tests/test_main.py
- ruby YAML parsing for CRD and sample ImpalaCluster manifests
- kubectl config current-context (k3d-impala-live)
- kubectl apply -f operator/impala-operator/manifests/crd-impalacluster.yaml
- kubectl apply -f operator/impala-operator/manifests/rbac.yaml
- kubectl create namespace impala-14983-live
- /tmp/impala-op-venv/bin/python reconcile driver invoking _ensure_namespace,
_ensure_ldap, and _ensure_impala with
spec.config.impalad.flags.num_reactor_threads=0 and
spec.config.impalad.queryDefaults={default_file_format=parquet,mt_dop=4}
- kubectl rollout status
deployment/impala-14983-live-impala-{statestored,catalogd,impalad,hms} -n
impala-14983-live
- kubectl get deployment impala-14983-live-impala-impalad -n impala-14983-live
-o jsonpath='{.spec.template.spec.containers[0].args}' (contains
-num_reactor_threads=0 and
-default_query_options=default_file_format=parquet,mt_dop=4)
- helm -n impala-14983-live status impala-14983-live (STATUS: deployed)
- kubectl get pods -n impala-14983-live (all core pods Running/Ready)
Implemented and validated manually; Cursor-assisted.
Change-Id: Ie24380d4700cf8377f291de2406cb6ed12f6d6f4
Co-authored-by: Cursor <[email protected]>
---
M docs/impala-on-kubernetes.md
M operator/impala-operator/main.py
M operator/impala-operator/manifests/crd-impalacluster.yaml
M operator/impala-operator/manifests/sample-impalacluster.yaml
A operator/impala-operator/tests/test_main.py
5 files changed, 304 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/65/24365/13
--
To view, visit http://gerrit.cloudera.org:8080/24365
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie24380d4700cf8377f291de2406cb6ed12f6d6f4
Gerrit-Change-Number: 24365
Gerrit-PatchSet: 13
Gerrit-Owner: Anubhav Jindal <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
