-
Notifications
You must be signed in to change notification settings - Fork 51
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: revert to ubuntu-20.04 for workflow jobs that fail on ubuntu-latest #4794
Conversation
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.
OK!
Thanks! Set MWP |
Hm, the system test build is getting stuck here:
We've seen these issues before, but never in CI, and I didn't think the change here would affect any of the other builds, but I'm looking into it. |
My guess is that the |
Problem: The ubuntu-latest image seems to have been recently updated to Ubuntu 22.04, which broke the python linting job, which now does't support python 3.6. Additionally, the system/coverage test build fails because docker-run-systest.sh apparently breaks on 22.04. Use runs-on: ubuntu-20.04 in the python-lint and ci-checks jobs to work around the failures for now.
2dfe0b6
to
67b80a4
Compare
Codecov Report
@@ Coverage Diff @@
## master #4794 +/- ##
==========================================
- Coverage 84.31% 83.46% -0.86%
==========================================
Files 412 415 +3
Lines 62547 70730 +8183
==========================================
+ Hits 52735 59033 +6298
- Misses 9812 11697 +1885
|
Oops, I had forgotten to remove MWP when adding ubuntu-20.04 to the |
Problem: The python linting job fails on ubuntu-latest because python 3.6 is no longer supported.
Use runs-on: ubuntu-20.04 in the python-lint job so that python 3.6 can still be used.
See actions/setup-python#355 (comment)