ci: add support for RHEL 9-based distributions #2007
+69
−4
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull request overview
This PR extends CI test coverage to include RHEL 9-based distributions (AlmaLinux 9, Rocky Linux 9, and Oracle Linux 9) across GitHub Actions workflows. The key change is the transition from PowerTools repository (RHEL 8) to CRB (CodeReady Builder) repository (RHEL 9) for installing test dependencies.
- Adds Rocky Linux 9 and AlmaLinux 9 to CI test matrices
- Configures RHEL 9-specific dependencies using CRB repository instead of PowerTools
- Installs required packages (swig, diffutils, gtest-devel, gmock-devel) via EPEL and CRB repositories
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/test_linux.yml | Adds Rocky Linux 9 to matrix and RHEL 9-based dependency configuration with CRB repository |
| .github/workflows/test_linux_py2.yml | Adds Rocky Linux 9 to matrix and RHEL 9-based dependency configuration with CRB repository |
| .github/workflows/trickops.yml | Adds dedicated Rocky Linux 9 test job with appropriate dependencies |
| .github/workflows/report_linux.yml | Adds Rocky Linux 9 to test report matrix |
| .github/workflows/report_linux_py2.yml | Adds Rocky Linux 9 to test report matrix |
| .github/workflows/alt_linux_distros.yml | Adds AlmaLinux 9 to matrix and RHEL 9-based dependency configuration with CRB repository |
| .github/workflows/report_alt_linux_distros.yml | Adds AlmaLinux 9 to test report matrix |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
|
thanks for the quick pr! just a few notes:
rebase out the changes to the other files and after you've made your fixes, force push the new changes |
Add Rocky Linux 9 and Oracle Linux 9 to the test and report workflows. Include RHEL 9-specific dependencies configuration with CRB repository enablement for gtest/gmock installation. Add dedicated trickops unit test job for Rocky Linux 9 to ensure compatibility with the newer RHEL version.
- Fix duplicate Oracle Linux 8 entry in test matrix, replace with Oracle Linux 9 - Add Oracle Linux 9 specific gtest installation steps using ol9_codeready_builder repository - Update Oracle Linux 8 gtest installation to be version-specific instead of OS-only match This ensures Oracle Linux 9 is properly tested with the correct dependency installation commands.
simtheverse
force-pushed
the
trick_issue_2005
branch
from
7eddcbb to
f93749f
Compare
Author
|
Done- let me know if that looks right |
- Removed `swig` from `tag_deps` (which tried to install it with other packages) - Added `swig` installation to the `conf_pkg` step after enabling the CRB repository - This ensures the CRB repository is enabled before attempting to install `swig`
Author
|
I see the CI errors, and I believe my new commits address them. |
Separated RHEL-based matrix configurations to prevent Rocky-specific commands from running on Oracle Linux and vice versa. Common RHEL dependencies remain shared while install_gtest commands are now distribution-specific (powertools for Rocky, codeready_builder for Oracle).
Oracle Linux 9 doesn't have a 'crb' repository - it uses 'ol9_codeready_builder'. Moved the crb enablement from generic RHEL 9 config to Rocky Linux 9 specific config to prevent "No matching repo to modify: crb" error.
- Install epel-release and update repos before enabling CRB on Rocky 9 - Add dnf-command(config-manager) plugin installation for Rocky 9 - Move swig from conf_pkg to tag_deps for Rocky 9 to ensure proper ordering - Add missing swig dependency to Oracle Linux 9 configuration These changes ensure package repositories are properly configured before attempting to enable additional repos and install dependencies, fixing build failures on Rocky Linux 9 and Oracle Linux 9 environments.
Add prerequisite steps to enable EPEL repository and install dnf config-manager before enabling ol9_codeready_builder repository. This ensures the required tools are available before attempting to configure additional repositories and install gtest dependencies on OracleLinux 9. Changes: - Install epel-release package - Update dnf package database - Install dnf-command(config-manager) plugin - Reorganize steps into conf_pkg and install_gtest sections
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Addresses #2005
Add AlmaLinux 9, Rocky Linux 9, and Oracle Linux 9 to the CI test matrix across all Linux workflow files. Configure RHEL 9-specific dependencies including:
This extends CI coverage to test against the latest RHEL 9 ecosystem alongside existing RHEL 8 support.