Repository navigation
fix(profiles): follow the CentOS Stream cloud image rename - #164
Conversation
CentOS Stream moved its composes to the "gen2" pungi configuration on 2026-08-19, which dropped the CentOS-Stream-GenericCloud-x86_64 image definition and kept only CentOS-Stream-GenericCloud: - centos-9-stream: https://gitlab.com/redhat/centos-stream/release-engineering/pungi-centos/-/commit/8bbb2eadd8b7e21d24ce63bb38b6aa9be520213f - centos-10-stream: https://gitlab.com/redhat/centos-stream/release-engineering/pungi-centos/-/commit/fa8f81e0c342b552a34d4be86fcf14228f99bca5 cloud.centos.org now publishes the latest images as CentOS-Stream-GenericCloud-<N>-latest.x86_64.qcow2, and the old CentOS-Stream-GenericCloud-x86_64-<N>-latest.x86_64.qcow2 URLs return HTTP 502, so every CentOS Stream target fails with an infrastructure error. Update the CentOS Stream 9 and 10 profiles, including the generated kernel-sweep ones, and the vm-images-extended Makefile target. Signed-off-by: Alban Crequy <albancrequy@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (7)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughCentOS Stream 9 and 10 image URLs now use the updated GenericCloud filenames. The change updates the extended image fetch target and affected VM profiles; cache destination names remain unchanged. The changelog records HTTP 502 errors from the previous URLs. ChangesCentOS Stream image URLs
Priority: ⬆️ High Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix · Severity of issue fixed: Medium Merge Risk: ⚪ Minimal · up to The updated image URLs are consistent across the fetch target and profiles, with cache names unchanged. No identified issue blocks merging. 🚥 Pre-merge checks | ✅ 7 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (7 passed)
Full details: Missing Regression TestsExplanation The PR changes observable image-download behavior but adds no regression test. The diff changes the CentOS Stream 9/10 URLs in Resolution Add an automated regression test that loads the CentOS Stream 9/10 profiles and asserts the exact
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
thanks for fixing merging it |
AIL:4
CentOS Stream switched its composes to the "gen2" pungi configuration on 2026-08-19 (c9s, c10s), which dropped the
CentOS-Stream-GenericCloud-x86_64image name. cloud.centos.org now servesCentOS-Stream-GenericCloud-<N>-latest.x86_64.qcow2, and the oldCentOS-Stream-GenericCloud-x86_64-<N>-latest.x86_64.qcow2URLs return HTTP 502.Since then every CentOS Stream target fails with
infra_error (download image: unexpected HTTP status 502 ...). For example, allcentos-stream-9-5.14cells in Inspektor Gadget's weekly compatibility run have been infrastructure errors since 2026-09-28: https://github.com/inspektor-gadget/inspektor-gadget/actions/runs/37271031045This updates the URL in the CentOS Stream 9/10 profiles (including the generated kernel-sweep ones) and in
make vm-images-extended.Summary by CodeRabbit