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
ci: Move from Ubuntu 18.04 to Ubuntu 20.04 Actions runners #14910
ci: Move from Ubuntu 18.04 to Ubuntu 20.04 Actions runners #14910
Conversation
Review period will end on 2023-03-07 at 21:51:09 UTC. |
Review period skipped due to |
Should we test on ubuntu 20.04? I think these tests were there to check if everything works fine on an older stack (glibc and so on), if I am not mistaken? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please switch these to 20.04 rather than removing them. We need to ensure we're testing a non-22.04 configuration so that we can ensure the brewed gcc/glibc path works. Thanks for the PR ❤️
- The 18.04 runners are deprecated and going away in less than a month. Today was another scheduled brownout from GitHub (https://github.blog/changelog/2022-08-09-github-actions-the-ubuntu-18-04-actions-runner-image-is-being-deprecated-and-will-be-removed-by-12-1-22/), and they've been unmarked as required checks in the branch protections because the images are unavailable and the jobs were failing. - However, we still need to test that Homebrew works correctly on non-latest Ubuntu versions so that we can test that we're correctly using brewed gcc/glibc, hence upgrading to 20.04 which is still old.
b08f6af
to
b95c549
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @issyl0! Can you toggle these back to required after this is merged?
Done! |
brew style
with your changes locally?brew typecheck
with your changes locally?brew tests
with your changes locally?