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 has been archived #234

Merged
merged 1 commit into from
Feb 10, 2022
Merged

CentOS 8 has been archived #234

merged 1 commit into from
Feb 10, 2022

Conversation

vchepkov
Copy link

@vchepkov vchepkov commented Feb 6, 2022

Pull Request (PR) description

CentOS 8 has been archived

gpgcheck: true
gpgkey: "file:///etc/pki/rpm-gpg/RPM-GPG-KEY-centosofficial"
target: '/etc/yum.repos.d/CentOS-AppStream.repo'
target: '/etc/yum.repos.d/CentOS-Linux-AppStream.repo'
Copy link
Member

Choose a reason for hiding this comment

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

is it a breaking change if we change the target? the old files could still exist and then we've the repo declared twice?

Copy link
Author

Choose a reason for hiding this comment

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

It is not, since this parameter is still not handled properly by yumrepo resource in any puppet versions
https://tickets.puppetlabs.com/browse/MODULES-7422

But, this is the state where the final Centos update is in

$ rpm -qf /etc/yum.repos.d/CentOS-Linux-AppStream.repo 
centos-linux-repos-8-3.el8.noarch

@vchepkov
Copy link
Author

vchepkov commented Feb 7, 2022

Seems I have permissions to merge, but since I never done it before want to make sure don't do it out of turn :)

@vchepkov vchepkov added the bug Something isn't working label Feb 10, 2022
@vchepkov vchepkov merged commit d5328a9 into voxpupuli:master Feb 10, 2022
@vchepkov vchepkov deleted the centos8 branch April 24, 2022 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants