Gokul Kolady has posted comments on this change. ( http://gerrit.cloudera.org:8080/24419 )
Change subject: IMPALA-15049: Harden Impala Kubernetes operator RBAC permissions ...................................................................... Patch Set 30: (1 comment) http://gerrit.cloudera.org:8080/#/c/24419/30/operator/impala-operator/tests/test_rbac_manifest.py File operator/impala-operator/tests/test_rbac_manifest.py: http://gerrit.cloudera.org:8080/#/c/24419/30/operator/impala-operator/tests/test_rbac_manifest.py@21 PS30, Line 21: import yaml One concern: this now imports yaml, but there’s no explicit dependency added in this CL. In a clean env this can fail with ModuleNotFoundError: No module named 'yaml'. Could we either declare PyYAML explicitly for this test path or refactor to avoid introducing a new dependency? -- To view, visit http://gerrit.cloudera.org:8080/24419 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia3eafc1f4ddcda423227ad5fc361e0bbbd4dad19 Gerrit-Change-Number: 24419 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, 05 Aug 2026 21:57:50 +0000 Gerrit-HasComments: Yes
