CSV update: use konflux repository for our images #557
Conversation
|
@spotlesstofu @gkurz - It's a pretty simple one, but I don't want to merge without a review :-) |
|
@littlejawa this change is reverted by running |
littlejawa
force-pushed
the
update_bundle_references_to_konflux
branch
from
60627f2 to
84c1aae
Compare
@gkurz - Right, I need to update the manager.yaml file too. |
Changing the image references to use the konflux repo. This is a temporary measure, to validate konflux's nudging process, where the bundle file gets updated when the images are rebuilt. We will put back the release registry URL when our release plan is set on the konflux side. Then konflux should be able to update the images with the release URL. Signed-off-by: Julien Ropé <[email protected]>
littlejawa
force-pushed
the
update_bundle_references_to_konflux
branch
from
84c1aae to
1ee4d5d
Compare
|
Updated again with the current reference for the operator - Build was failing because the image didn't exist anymore. |
|
@littlejawa: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
There was a problem hiding this comment.
Pull Request Overview
This PR implements temporary updates to image references for the operator, switching from the Red Hat registry to the konflux repository to validate the nudging process during image rebuilds.
- Updated image references and digests in config/manager/manager.yaml
- Revised image references, digests, and operator image in the CSV manifest to align with the konflux repository
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| config/manager/manager.yaml | Image references updated to konflux images with new digests |
| bundle/manifests/sandboxed-containers-operator.clusterserviceversion.yaml | CSV updated to reference konflux operator images and related images with new digests |
Comments suppressed due to low confidence (2)
config/manager/manager.yaml:80
- Confirm that the new image digest for the 'osc-monitor' image is correct and aligns with the konflux repository build output.
value: quay.io/redhat-user-workloads/ose-osc-tenant/osc-monitor@sha256:1e19738be039443b1086a22c446897216e791a6cd72da5aeeb6e538f1c9aa447
bundle/manifests/sandboxed-containers-operator.clusterserviceversion.yaml:397
- Ensure that the updated operator image reference is consistent with the other image changes and that its digest reflects the intended temporary measure.
image: quay.io/redhat-user-workloads/ose-osc-tenant/osc-operator@sha256:7aecfb575e553ec53ccb1880e2f77bb66f48190c49372723882a3ca7d6ba315c
Changing the image references to use the konflux repo. This is a temporary measure, to validate konflux's nudging process, where the bundle file gets updated when the images are rebuilt.
We will put back the release registry URL when our release plan is set on the konflux side. Then konflux should be able to update the images with the release URL.
For reference: I am following the instructions from https://konflux-ci.dev/docs/advanced-how-tos/building-olm/#automating-updates-for-image-references-in-the-csv