Disable local-storage-operator and ptp-operator for 5.0 ec.2#10724
Conversation
Both bundles fail Konflux Enterprise Contract validation during prepare-release-konflux for 5.0 ec.2: - local-storage-operator: olm.unpinned_references on the CSV must-gather-image annotation (registry.redhat.io/openshift4/ ose-local-storage-mustgather-rhel9 with floating tag). - ptp-operator: olm.allowed_registries and olm.unpinned_references on the CSV must-gather-image annotation (quay.io/openshift/ origin-ptp-operator-must-gather:4.21). Upstream alignment fixes are in flight (must-gather refs in the CSV must match the from.name in image-references so doozer's _replace_image_references can substitute them with digest-pinned production pullspecs): - openshift/local-storage-operator#624 - openshift/ptp-operator#699 Both images have for_payload: false so disabling here does not affect the release payload; it only excludes them from the bundle/ FBC sweep for ec.2. Re-enable once the upstream PRs merge and the bundles build cleanly.
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Hi @redhat-chai-bot. Thanks for your PR. I'm waiting for a openshift-eng member to verify that this patch is reasonable to test. If it is, they should reply with Tip We noticed you've done this a few times! Consider joining the org to skip this step and gain Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions 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. |
Adds
mode: disabledtoimages/local-storage-operator.ymlandimages/ptp-operator.ymlso the LSO and PTP operator + bundle builds are skipped during 5.0 ec.2prepare-release-konflux.Why
Both bundles fail Konflux Enterprise Contract (EC) validation during prepare-release-konflux for 5.0 ec.2:
olm.unpinned_referenceson the CSVmust-gather-imageannotation (registry.redhat.io/openshift4/ose-local-storage-mustgather-rhel9:v4.22.0is a floating tag).olm.allowed_registriesandolm.unpinned_referenceson the CSVmust-gather-imageannotation (quay.io/openshift/origin-ptp-operator-must-gather:4.21is from a disallowed upstream registry and floating).Root cause is that the must-gather reference in the CSV doesn't match the
from.namedeclared inimage-references, so doozer's_replace_image_referencesdoesn't substitute it with the digest-pinned production pullspec at bundle build time.Upstream alignment fixes are in flight:
Safety
Both images have
for_payload: false, so disabling them does not affect the release payload (no risk of the same payload-consistency error we hit earlier withaws-node-termination-handler). They ship as OLM bundles only — disabling here just excludes them from the bundle/FBC sweep for ec.2.Follow-up
Revert this disable once the upstream PRs merge and the bundles build cleanly through Konflux EC.
Context
Opened on behalf of the ART team from <https://redhat-internal.enterprise.slack.com/archives/C0ARQM5UV88/p1779227614610569|this Slack thread>.