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

add support for EL9 (RedHat, Rocky and AlmaLinux) #182

Merged
merged 1 commit into from
Feb 21, 2023
Merged

add support for EL9 (RedHat, Rocky and AlmaLinux) #182

merged 1 commit into from
Feb 21, 2023

Conversation

sircubbi
Copy link
Contributor

This PR will add support for EL9 and also include AlmaLinux in the process.

Please note that CI is currently failing (which I fixed in PR #181). I can rebase once that other PR got merged.

@nward
Copy link
Collaborator

nward commented Feb 21, 2023

Haha - beat me to this one as well!

Does Alma have the same paths for FreeRADIUS as CentOS/RHEL/Rocky? I've not used Alma.

@sircubbi
Copy link
Contributor Author

Does Alma have the same paths for FreeRADIUS as CentOS/RHEL/Rocky? I've not used Alma.

It actually does. Only difference between the different flavours are more or less the copyrighted artwork and logos from RedHat.

@nward
Copy link
Collaborator

nward commented Feb 21, 2023

Have you had a look at #178? I think the same issue exists in EL9

@sircubbi
Copy link
Contributor Author

Have you had a look at #178? I think the same issue exists in EL9

Yes, I saw that. However it is not necessary in EL9, since bootstrap is not called there. Comparing the systemd-Unitfiles I see that ExecStartPre=-/bin/sh /etc/raddb/certs/bootstrap is no longer present on EL9, most likely related to this changelog-entry:

* Wed Mar 10 2021 Robbie Harwood <rharwood@redhat.com> - 3.0.21-11
- Disable automatic bootstrap

I just rolled out a new freeradius-configuration on Rocky9 yesterday and had no issues with this puppet module (except the missing guess for the version of the freeradius-package, which is fixed with this PR).

@nward
Copy link
Collaborator

nward commented Feb 21, 2023

Please note that CI is currently failing (which I fixed in PR #181). I can rebase once that other PR got merged.

Go for gold

@sircubbi
Copy link
Contributor Author

Please note that CI is currently failing (which I fixed in PR #181). I can rebase once that other PR got merged.
Go for gold

Thanks, rebase is done and the CI now actually goes green ;)

@nward nward merged commit 0159cae into djjudas21:main Feb 21, 2023
@sircubbi sircubbi deleted the add_el9_support branch February 21, 2023 08:45
@djjudas21
Copy link
Owner

Thanks @nward and @sircubbi for your efforts on this 👍

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.

3 participants