Gokul Kolady has posted comments on this change. ( http://gerrit.cloudera.org:8080/24362 )
Change subject: IMPALA-14980: Add operator deploy and sample CR ...................................................................... Patch Set 33: (2 comments) http://gerrit.cloudera.org:8080/#/c/24362/33/operator/impala-operator/manifests/deployment.yaml File operator/impala-operator/manifests/deployment.yaml: http://gerrit.cloudera.org:8080/#/c/24362/33/operator/impala-operator/manifests/deployment.yaml@38 PS33, Line 38: image: ghcr.io/example/impala-operator:latest Quick clarification: this manifest currently uses ghcr.io/example/impala-operator:latest. Is that intentionally a placeholder for docs/smoke flow, or should this be a concrete published tag/digest for reproducible installs? http://gerrit.cloudera.org:8080/#/c/24362/33/operator/impala-operator/manifests/deployment.yaml@39 PS33, Line 39: imagePullPolicy: IfNotPresent Minor reproducibility nit: :latest with IfNotPresent can lead to stale/non-deterministic operator versions across environments. If this is meant to be a stable baseline manifest, would pinning an immutable tag/digest be better? -- To view, visit http://gerrit.cloudera.org:8080/24362 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifb334135dfd333dd5b14756ec8f41551981677e6 Gerrit-Change-Number: 24362 Gerrit-PatchSet: 33 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: Fri, 26 Jun 2026 19:56:27 +0000 Gerrit-HasComments: Yes
