Add Rocky Linux 10 to CI and package builds.#20578
Conversation
thiagoftsm
left a comment
There was a problem hiding this comment.
I am approving this PR based on the CI results. LGTM!
There was a problem hiding this comment.
Pull Request Overview
This PR extends CI and package build support to include RHEL 10, Alma Linux 10, and Rocky Linux 10 by updating various distro configuration entries.
- Updates the Rocky Linux configuration from version "9" to "10" and adds a new base_image reference.
- Adjusts the repo_distro and alt_links values for Rocky Linux to match the new version.
- Updates the RHEL configuration from version "9.x" to "10.x" and maintains an alias for the previous version.
Comments suppressed due to low confidence (3)
.github/data/distros.yml:301
- It is unclear why there is a separate rocky entry for version '9' after updating the main rocky block to version '10'. Consider adding a comment to clarify the purpose of maintaining legacy configuration.
- <<: *rocky
.github/data/distros.yml:306
- The 'repo_distro' remains set to 'el/9' in the legacy block despite the primary configuration updating to 'el/10'. If this discrepancy is intentional for legacy support, please add a comment explaining the rationale.
repo_distro: el/9
.github/data/distros.yml:281
- [nitpick] Consider adding a comment explaining why a fully qualified image URL is used for Rocky Linux 10, whereas the legacy entry for version '9' uses a shorthand notation.
base_image: "quay.io/rockylinux/rockylinux:10"
|
Hi @Ferroin, packages don't seem to appear in the public repository yet. Do you know when it's going to appear in the repo? I also posted a question in Discord under #general |
It will show up in the stable release channel once we actually create a stable release with these changes. I don’t recall if we have a concrete date yet, but we should have a new minor release soon, and this will be included in that release. |
Summary
This provides package builds for RHEL 10, Alma Linux 10, and Rocky Linux 10.
Test Plan
The new CI jobs pass on this PR.