TRT-2545: Extend known image checker to cover openshift-must-gather-* namespaces#30777
TRT-2545: Extend known image checker to cover openshift-must-gather-* namespaces#30777elfosardo wants to merge 1 commit intoopenshift:mainfrom
Conversation
|
Pipeline controller notification For optional jobs, comment This repository is configured in: automatic mode |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: elfosardo, smg247 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Scheduling required tests: |
|
@elfosardo: This pull request references TRT-2545 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the ticket to target the "4.22.0" version, but no target version was set. DetailsIn response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
|
we need #30784 first |
|
/retest |
1 similar comment
|
/retest |
The known image checker validates that tests only pull images from approved sources (cluster payload, mirrored repos, internal registry) to ensure disconnected environment compatibility. Previously it only checked e2e-* namespaces, so images pulled by oc adm must-gather in openshift-must-gather-* namespaces were never validated. This gap allowed operator-discovered must-gather images (e.g. cnv-must-gather-rhel9 from registry.redhat.io) to be pulled without detection, which would break disconnected clusters and caused pathological ImagePullBackOff events on IPv6-only clusters where registry.redhat.io is unreachable.
a6890ba to
3c5f4ee
Compare
|
New changes are detected. LGTM label has been removed. |
|
Scheduling required tests: |
|
/retest |
3 similar comments
|
/retest |
|
/retest |
|
/retest |
|
@elfosardo: The following tests failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
|
Job Failure Risk Analysis for sha: 3c5f4ee
|
|
/retest |
1 similar comment
|
/retest |
The known image checker validates that tests only pull images from approved sources (cluster payload, mirrored repos, internal registry) to ensure disconnected environment compatibility. Previously it only checked e2e-* namespaces, so images pulled by oc adm must-gather in openshift-must-gather-* namespaces were never validated.
This gap allowed operator-discovered must-gather images (e.g. cnv-must-gather-rhel9 from registry.redhat.io) to be pulled without detection, which would break disconnected clusters and caused pathological ImagePullBackOff events on IPv6-only clusters where registry.redhat.io is unreachable.