Skip to content

Conversation

ztmr
Copy link

@ztmr ztmr commented Aug 11, 2023

Starting from AlmaLinux 8.5, the repository filename is called almalinux-powertools.repo instead of almalinux-PowerTools.repo.

This change should fix that.

@Brueggus
Copy link

Brueggus commented Dec 2, 2023

@robertdebock Could you take this PR into consideration?
see my comment below

This role is currently broken on Alma Linux:

RUNNING HANDLER [robertdebock.powertools : Yum update cache] ********************************************************************************************************************************************************************************
fatal: [REDACTED]: FAILED! => {"changed": false, "msg": "Cannot find a valid baseurl for repo: powertools", "rc": 1, "results": []}

vars/main.yml Outdated
_powertools_repo_file:
RedHat: "{{ ansible_distribution }}-PowerTools.repo"
Rocky: "{{ ansible_distribution }}-PowerTools.repo"
Rocky: "{{ ansible_distribution|lower }}-PowerTools.repo"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ztmr This change appears to be wrong, Rocky Linux uses uppercase letters, see https://rockylinux.pkgs.org/8/rockylinux-baseos-x86_64/rocky-repos-8.9-1.6.el8.noarch.rpm.html => Files

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Brueggus for Rocky 9.x it is lowercase, for Rocky 8.x it is camelcase, what a mess. I've fixed this elsewhere already and didn't sync this change back. I'll try to update this PR accordingly later this week.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Brueggus I take the earlier comment back, I got it confused with the change in Alma mid 8.x -- you were right, the fix commit for 9.x broke 8.x.

Is the updated version working for you now?

@ztmr ztmr force-pushed the fix/almalinux-repo-filename-change-8-5-plus branch from 7e430e9 to 5e93ea7 Compare December 5, 2023 19:00
@ztmr ztmr force-pushed the fix/almalinux-repo-filename-change-8-5-plus branch from c9a935a to e195f9e Compare December 5, 2023 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants