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

CentOS7: default to EPEL7 as source #983

Merged
merged 1 commit into from
May 19, 2024
Merged

Conversation

bastelfreak
Copy link
Member

@bastelfreak bastelfreak commented May 19, 2024

Historically we used the garethr/erlang module as soft dependency on CentOS 7. This also configured the EPEL7 repository. I always always a bit unsure where the actual rabbitmq packages come from. I assumed somehow from the erlang repo. My impression is that the erlan repo isn't used at all. rabbitmq packages where always pulled from EPEL7. The problem with the garethr/erlang is that it depends on the deprecated stahnma/epel (which is now puppet/epel) and uses topscope facts and
variables. The module doesn't work anymore with Puppet 8.

This PR replaces the unused garethr/erlang directly with the EPEL module.

@bastelfreak bastelfreak force-pushed the epel branch 5 times, most recently from 16f3435 to c9c40a4 Compare May 19, 2024 15:57
@bastelfreak bastelfreak changed the title CentOS7: Install EPEL or upstream repos CentOS7: default to EPEL7 as source May 19, 2024
@bastelfreak bastelfreak added the bug Something isn't working label May 19, 2024
@@ -505,6 +507,8 @@
default: {
}
}
} elsif ($facts['os']['family'] == 'RedHat' and $facts['os']['release']['major'] == '7') {
require epel
Copy link
Member Author

Choose a reason for hiding this comment

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

I don't have a strong opinion on how we should call the epel module. We could also just add it to the acceptance tests or introduce another parameter for it.

Historically we used the garethr/erlang module as soft dependency on
CentOS 7. This also configured the EPEL7 repository. I always always a
bit unsure where the actual rabbitmq packages come from. I assumed
somehow from the erlang repo. My impression is that the erlan repo isn't
used at all. rabbitmq packages where always pulled from EPEL7. The
problem with the garethr/erlang is that it depends on the deprecated
`stahnma/epel` (which is now puppet/epel) and uses topscope facts and
variables. The module doesn't work anymore with Puppet 8.

This PR replaces the unused garethr/erlang directly with the EPEL
module.
@wyardley
Copy link
Contributor

The actual rmq packages used to come from packagecloud, and then we switched to them coming from the OS vendor repo. They did not ever come from the Erlang repo, but I believe they had a dependency on some version of Erlang being installed that perhaps the package itself did not satisfy? This was ancient history, though.

this makes sense to me if it works, though may make sense to ship the other, safer changes first since there are a bunch.

@bastelfreak
Copy link
Member Author

@wyardley you want to release 13.3.0 first, then merge #983 and do another release?

@wyardley
Copy link
Contributor

If #979 looks good to you as-is, let's do that; also, is it possible that CentOS 8 (see #900) will now work, though based on https://en.wikipedia.org/wiki/CentOS, seems like CentOS 8 actually was already EOLed a long time ago, and 7 will be next month?

@bastelfreak
Copy link
Member Author

We already started dropping CentOS 7 in some repos (https://voxpupuli.org/blog/2024/05/08/CentOS-7-CentOS-Stream-8-deprecation/) but it would be nice to have one release where it's actually fixed. And the updated dependencies will make it easier for users to upgrade.

@bastelfreak bastelfreak merged commit 81e5057 into voxpupuli:master May 19, 2024
6 checks passed
@bastelfreak bastelfreak deleted the epel branch May 19, 2024 17:03
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.

3 participants