-
-
Notifications
You must be signed in to change notification settings - Fork 348
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
Attempt to fix PyPy CI #2536
Attempt to fix PyPy CI #2536
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #2536 +/- ##
=======================================
Coverage 92.45% 92.45%
=======================================
Files 118 118
Lines 16351 16352 +1
Branches 3156 3156
=======================================
+ Hits 15118 15119 +1
Misses 1104 1104
Partials 129 129
|
I wonder if this is happening consistently.. Does faulthandler work under PyPy? In some cases, I've found that pytest was crashing due to an old version of its |
By the way, I've just looked into Oh, and it's not clear why not use PyPy that GHA ships for testing... |
retriggered to verify: https://github.com/python-trio/trio/actions/runs/3945507679/jobs/6766090599. |
I was simply updating |
I just saw it outside this PR. |
I've taken a stab at it #2541 |
It looks like the failure in I will look at the rest of the errors at a later time. |
OK I think this is ready for another review. The workflows will error but they shouldn't block required checks. We can figure those out at a later date. |
Looks like all that needs to happen is macos needs its timeout bumped to like 20 minutes (just like windows) https://github.com/python-trio/trio/actions/runs/4169791936/jobs/7218120369 |
Both MacOS tests timed out on Although: it also fails here (kinda): https://github.com/python-trio/trio/actions/runs/4221075009/jobs/7327960276#step:5:1100 |
9afa0f4
to
ae6beaa
Compare
I forgot to undo this while switching how nightlys are installed.
I filed an issue about our GC behavior over at https://foss.heptapod.net/pypy/pypy/-/issues/3900; hopefully they can fix our problem or tell us what we're missing. |
This should in theory enable the already defined PyPy runs on MacOS and Windows. They never worked!!
This also bumps the
@actions/setup-python
version we use (the old one errored for nightly PyPy versions & gave us a warning in action runs due to using node 12 vs 16)