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

Switch to Alma8 for testsuite #564

Merged
merged 2 commits into from
Nov 27, 2024
Merged

Switch to Alma8 for testsuite #564

merged 2 commits into from
Nov 27, 2024

Conversation

atoomic
Copy link
Collaborator

@atoomic atoomic commented Nov 26, 2024

We originally used a CentOS 7 container. But it cannot be used to run native GitHub Actions relying on node20.
With this change we bump the container to use a more modern Almalinux 8 flavor, which is used for running our unit tests.

Note that we preserve the e2e test using a self-hosted runner relying on C7 VMs.

@atoomic atoomic force-pushed the testsuite branch 8 times, most recently from af51772 to 89aabdd Compare November 27, 2024 00:20
@atoomic atoomic changed the title pin node 16 Switch to Alma8 for testsuite Nov 27, 2024
@atoomic atoomic force-pushed the testsuite branch 3 times, most recently from faada66 to 6fb9c9e Compare November 27, 2024 01:23
@atoomic atoomic requested a review from toddr November 27, 2024 01:23
We are using a cPanel Perl stack, thus we only
need to list missing required packages.

We do not really need to run cpm/cpanm twice.
@toddr toddr merged commit 5aacc02 into main Nov 27, 2024
19 of 20 checks passed
@toddr toddr deleted the testsuite branch November 27, 2024 05:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants