-
-
Notifications
You must be signed in to change notification settings - Fork 559
Fix RHEL 8 repository names for perl-IPC-Run package installation across all variants #796
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: damacus <[email protected]>
Slowest examplesTop 10 slowest examples (0.01 seconds, 49.23% of total time)
|
options ['--enablerepo=ol8_codeready_builder'] | ||
else | ||
options('--enablerepo=powertools') | ||
options('--enablerepo=codeready-builder-for-rhel-8') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot This is only correct for RHEL, powertools is required for centos-stream, almalinux and rocky.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're absolutely right! I've updated the code to handle all RHEL 8 variants correctly. The new implementation uses case node['platform']
to detect the specific platform:
- RHEL:
codeready-builder-for-rhel-8
- Oracle Linux:
ol8_codeready_builder
- CentOS Stream, AlmaLinux, Rocky:
powertools
(restored for these platforms)
This preserves the original powertools
repository for CentOS Stream, AlmaLinux, and Rocky Linux while fixing the issue for actual RHEL systems. Commit: 1adde8a
Co-authored-by: ramereth <[email protected]>
The
postgresql_role
resource was failing on RHEL 8.10 with an "Unknown repo: 'powertools'" error when attempting to install theperl-IPC-Run
package dependency required for the PostgreSQLpg
gem.Problem
The issue occurred in
libraries/sql/_connection.rb
where the code was trying to installperl-IPC-Run
from thepowertools
repository:However, the
powertools
repository doesn't exist on RHEL 8 systems, causing the package installation to fail and breaking thepostgresql_role
functionality.Solution
Updated the code to use platform-specific repository names for different RHEL 8 variants:
This ensures each RHEL 8 variant uses its correct CodeReady Builder repository:
codeready-builder-for-rhel-8
ol8_codeready_builder
powertools
Impact
postgresql_role
functionality on RHEL 8.10 and other RHEL 8 variantscrb
repository)This change provides comprehensive support for all RHEL 8 variants by detecting the specific platform and using the appropriate repository name.
Fixes #783.
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.