You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Anubhav Jindal (Code Review)" <ge...@cloudera.org> on 2026/07/06 21:44:05 UTC

[Impala-ASF-CR] IMPALA-14981: Add operator reconciler packaging

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/24363

to look at the new patch set (#35).

Change subject: IMPALA-14981: Add operator reconciler packaging
......................................................................

IMPALA-14981: Add operator reconciler packaging

Add the operator reconciliation implementation and packaging assets needed
to run chart-backed Impala and optional LDAP reconciliation.

Handle LDAP disable transitions declaratively by uninstalling an existing
LDAP release when ldapEnabled is set to false.

Add ASF license headers to the new operator source and packaging files.

Testing:
- python3 -m py_compile operator/impala-operator/main.py
- 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-14981-live
- /tmp/impala-op-venv/bin/python reconcile driver invoking _ensure_namespace, _ensure_ldap, and _ensure_impala in operator/impala-operator/main.py with spec.set persistence.accessModes[0]=ReadWriteOnce
- kubectl rollout status deployment/impala-14981-live-impala-{statestored,catalogd,impalad,hms} -n impala-14981-live
- helm -n impala-14981-live status impala-14981-live (STATUS: deployed)
- kubectl get pods -n impala-14981-live (all core pods Running/Ready)
- helm create /tmp/ldap-dummy-14981 && helm upgrade --install impala-ldap-14981-live /tmp/ldap-dummy-14981 -n impala-14981-live
- /tmp/impala-op-venv/bin/python reconcile driver invoking _ensure_ldap with ldapEnabled=false and ldapReleaseName=impala-ldap-14981-live
- helm -n impala-14981-live status impala-ldap-14981-live (fails after operator path; release removed)

Change-Id: I9b66e32b00724221ead2f4975802a26b2be7e248
Assisted-by: GPT-5.3 (Cursor)
Co-authored-by: Cursor <cu...@cursor.com>
---
A operator/impala-operator/Dockerfile
A operator/impala-operator/main.py
A operator/impala-operator/requirements.txt
3 files changed, 369 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/63/24363/35
-- 
To view, visit http://gerrit.cloudera.org:8080/24363
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9b66e32b00724221ead2f4975802a26b2be7e248
Gerrit-Change-Number: 24363
Gerrit-PatchSet: 35
Gerrit-Owner: Anubhav Jindal <an...@gmail.com>
Gerrit-Reviewer: Abhishek Rawat <ar...@cloudera.com>
Gerrit-Reviewer: Anubhav Jindal <an...@gmail.com>
Gerrit-Reviewer: Gokul Kolady <go...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com>