Anubhav Jindal has posted comments on this change. ( http://gerrit.cloudera.org:8080/24362 )
Change subject: IMPALA-14980: Add operator deploy and sample CR ...................................................................... Patch Set 34: (2 comments) Done 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: # Placeholder image reference for bootstrap/demo flow. > Quick clarification: this manifest currently uses ghcr.io/example/impala-op Good point, thanks. This manifest is meant as bootstrap/sample flow, and in this patchset I clarified that real installs should override the image with a concrete published tag/digest. http://gerrit.cloudera.org:8080/#/c/24362/33/operator/impala-operator/manifests/deployment.yaml@39 PS33, Line 39: # Override with a concrete published tag or digest for real installs. > Minor reproducibility nit: :latest with IfNotPresent can lead to stale/non- Agreed. I updated the manifest to avoid :latest by using a deterministic placeholder tag and added guidance to pin an immutable tag/digest for reproducible installs. -- 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: 34 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: Mon, 06 Jul 2026 22:01:28 +0000 Gerrit-HasComments: Yes
