Mirror the pause image to additional locations#78321
Mirror the pause image to additional locations#78321mgencur wants to merge 2 commits intoopenshift:mainfrom
Conversation
…__nK-VRxiifU0Z The image index is now different: https://github.com/openshift/origin/blob/main/vendor/k8s.io/kubernetes/test/utils/image/manifest.go Some time ago, some images were removed from the list so the resulting index is 22, not 25. We now mirror the image to both 22 and 25 to preserve backwards compatibility.
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: mgencur 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 |
|
/pj-rehearse periodic-ci-openshift-hypershift-release-4.22-periodics-mce-e2e-agent-disconnected-ovn-dualstack-metal-conformance |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: openshift/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (3)
💤 Files with no reviewable changes (2)
WalkthroughThree CI/CD workflow configuration files are modified: one adds an image-mirroring step for pause image handling in Hypershift conformance setup, while two remove wait step references from MCE workflow post-test cleanup sequences. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 12✅ Passed checks (12 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
@mgencur: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
[REHEARSALNOTIFIER]
A total of 603 jobs have been affected by this change. The above listing is non-exhaustive and limited to 25 jobs. A full list of affected jobs can be found here Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
1 participant
Fixes the issue from #78238 (comment)
The image index is now different:
https://github.com/openshift/origin/blob/main/vendor/k8s.io/kubernetes/test/utils/image/manifest.go
Some time ago, some images were removed from the list so the resulting
index is 22, not 25. We now mirror the image to both 22 and 25 to preserve backwards
compatibility.
This is a temporary solution. It needs some re-work to prevent the issue in the future as it already happens 3 times:
e.g. e4b3139
Summary by CodeRabbit
Tests
Chores