Anubhav Jindal has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24363 )

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


Patch Set 30:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/24363/28/operator/impala-operator/main.py
File operator/impala-operator/main.py:

http://gerrit.cloudera.org:8080/#/c/24363/28/operator/impala-operator/main.py@52
PS28, Line 52:     "credential",
> We log full Helm command lines, including all --set values from spec.set. T
Implemented redaction for sensitive Helm --set/--set-string values in operator 
logs and error messages


http://gerrit.cloudera.org:8080/#/c/24363/28/operator/impala-operator/main.py@110
PS28, Line 110:     if stderr:
> auth.ldap.bindPattern can contain commas (e.g. cn=#UID,dc=example,dc=org), 
Thank you, LDAP URI/bindPattern now use --set-string with escaping so 
comma-containing bind patterns are preserved


http://gerrit.cloudera.org:8080/#/c/24363/28/operator/impala-operator/main.py@209
PS28, Line 209:         target_ns,
> We currently trust spec.namespace as the target namespace. Since ImpalaClus
Fixed, updated operator behavior to always reconcile/delete in 
metadata.namespace and ignore mismatched spec.namespace



--
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: comment
Gerrit-Change-Id: I9b66e32b00724221ead2f4975802a26b2be7e248
Gerrit-Change-Number: 24363
Gerrit-PatchSet: 30
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]>
Gerrit-Comment-Date: Wed, 17 Jun 2026 22:34:18 +0000
Gerrit-HasComments: Yes

Reply via email to