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 out of box support for Rocky Linux #205

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bluikko
Copy link
Contributor

@bluikko bluikko commented Jan 13, 2025

Closes #204.

Add Rocky Linux 8 and 9 OS string and OVAL URLs.

Currently untested.

I notice that in DefaultConfig.php https://github.com/CESNET/pakiti-server/blob/master/src/common/DefaultConfig.php#L147 there is a regex for RHEL that includes all variants, including AlmaLinux that already has the 2 changes included in this PR.
Wondering if that is normal or should AlmaLinux and Rocky Linux be removed from lines 147-148 if they are handled individually later in the same file?

@HavrilaJ
Copy link
Contributor

Hi, no that Alma shouldn't be with RHEL becasue it has its own OVAL right now. I will fix it in DefaultConfig.

I didn't know that Rocky published its own OVALs. I just swiflty tried it, but pakiti has some errors reading it, but as soon as I will find out where is the problem it will be added here.

@bluikko
Copy link
Contributor Author

bluikko commented Jan 13, 2025

Hi, no that Alma shouldn't be with RHEL becasue it has its own OVAL right now. I will fix it in DefaultConfig.

The same 2 lines also include Rocky Linux that this PR seeks to address.

I didn't know that Rocky published its own OVALs. I just swiflty tried it, but pakiti has some errors reading it, but as soon as I will find out where is the problem it will be added here.

Thanks!

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.

Rocky Linux out of box support
2 participants