Skip to content

Conversation

stv0g
Copy link
Contributor

@stv0g stv0g commented Sep 8, 2025

No description provided.

@stv0g stv0g requested a review from n-eiling as a code owner September 8, 2025 10:39
@n-eiling
Copy link
Member

n-eiling commented Sep 8, 2025

Why are you doing this? I think Rocky 9 might still be used at ACS, or not? I wouldn't remove a distribution that is actively being used for running VILLASnode.

@stv0g
Copy link
Contributor Author

stv0g commented Sep 8, 2025

I see, I've updated the distro version in an attempt to fix a problem with broken Linux headers.

But also to improve the experience for first time users, who start working on VILLASnode using Rocky 10.

I think I will add both versions to the CI, if there are still users of Rocky 9.

@n-eiling
Copy link
Member

n-eiling commented Sep 9, 2025

Yes, I'm always for adding more recent versions. I just want us to be cautious about dropping older versions, because this essentially means we are dropping support for these distributions. CI runtimes shouldn't be an issue because builds should be able to run in parallel, right?

@stv0g stv0g marked this pull request as draft September 9, 2025 13:42
@stv0g stv0g changed the title Update RockyLinux docker image Add RockyLinux 10 builds to CI Sep 9, 2025
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