Skip to content
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

CentOS 8 PowerTools repo needs to be lowercase in newer kernel versions #831

Merged
merged 1 commit into from
Dec 9, 2020

Conversation

enrico-usai
Copy link
Contributor

@enrico-usai enrico-usai commented Dec 7, 2020

We're enabling PowerTools or powertools according to the kernel version:
https://access.redhat.com/articles/3078#RHEL8

The name changed with 8.3 release:
https://bugzilla.redhat.com/show_bug.cgi?id=1900785

Additional changes

I changed the find_rhel_minor_version to return just a number, without the . character to be easier to compare it.

With this patch I'm also removing epel-release installation which is already performed as a packer step.

@rexcsn rexcsn force-pushed the wip/centos8-power branch from b067b04 to e909e43 Compare December 7, 2020 21:20
We're enabling `PowerTools` or `powertools` according to the kernel version:
https://access.redhat.com/articles/3078#RHEL8

The name changed with 8.3 release:
https://bugzilla.redhat.com/show_bug.cgi?id=1900785

## Additional changes

I changed the `find_rhel_minor_version` to return just a number,
without the `.` character to be easier to compare it.

With this patch I'm also removing `epel-release` installation
which is already performed as a packer step.

Signed-off-by: Enrico Usai <[email protected]>
@enrico-usai enrico-usai changed the title CentOS 8 PowerTools repo needs to be lowercase CentOS 8 PowerTools repo needs to be lowercase in newer kernel versions Dec 9, 2020
@enrico-usai enrico-usai merged commit 690368b into aws:develop Dec 9, 2020
@enrico-usai enrico-usai deleted the wip/centos8-power branch December 9, 2020 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants