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

remove continue-on-error from free-threaded CI job #4546

Merged
merged 4 commits into from
Sep 23, 2024

Conversation

ngoldbaum
Copy link
Contributor

Now that 3.13.0rc2 is out, the tests shouldn't be crashing anymore.

In principle it's also possible internal uses of GILOnceCell could also cause crashes. In practice I don't ever see any crashes caused by GILOnceCell in my local testing.

I do need help with one thing, there is currently a failing test introduced by #4453, which added a new diagnostic warning to compiler errors if the abi3 feature is enabled. But since the free-threaded build (like pypy) simply warns that abi3 isn't supported and ignores it, this leads to a test failure due to this new line in the compiler error:

$ diff -u expected actual
--- expected	2024-09-11 14:26:36
+++ actual	2024-09-11 14:26:47
@@ -6,6 +6,7 @@
   |
   = help: the trait `PyClassBaseType` is not implemented for `PyBool`
   = note: `PyBool` must have `#[pyclass(subclass)]` to be eligible for subclassing
+  = note: with the `abi3` feature enabled, PyO3 does not support subclassing native types
   = help: the following other types implement trait `PyClassBaseType`:
             PyAny
             PyArithmeticError
@@ -30,6 +31,7 @@
   |
   = help: the trait `PyClassBaseType` is not implemented for `PyBool`
   = note: `PyBool` must have `#[pyclass(subclass)]` to be eligible for subclassing
+  = note: with the `abi3` feature enabled, PyO3 does not support subclassing native types
   = help: the following other types implement trait `PyClassBaseType`:
             PyAny
             PyArithmeticError

I'm not sure how to account for that in the compiler error tests.

@ngoldbaum ngoldbaum added CI-build-full CI-skip-changelog Skip checking changelog entry labels Sep 11, 2024
@davidhewitt
Copy link
Member

Nice! I pushed a commit to fix the diagnostic (realised it was easier to push than to describe).

@davidhewitt
Copy link
Member

Looks like there's a (new?) failure in test_gc 🤔

@ngoldbaum ngoldbaum mentioned this pull request Sep 20, 2024
3 tasks
@ngoldbaum
Copy link
Contributor Author

I can reproduce the failure on an x86 linux machine. I wonder why it only happens on x86 linux so far, I'll try to understand more.

@ngoldbaum
Copy link
Contributor Author

I ignored the assert that was failing, I've convinced myself it is not a valid thing to assert on the free-threaded build and left a comment to that effect.

I don't see any other crashes on linux besides python/cpython#124375 (I ran into and reported it working on this PR), which I think is unrelated to PyO3.

Copy link
Member

@davidhewitt davidhewitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, thanks. Think there's just one thing which might have changed too far...

.github/workflows/ci.yml Show resolved Hide resolved
Co-authored-by: David Hewitt <mail@davidhewitt.dev>
@davidhewitt davidhewitt added this pull request to the merge queue Sep 23, 2024
Merged via the queue into PyO3:main with commit eccff58 Sep 23, 2024
83 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-build-full CI-skip-changelog Skip checking changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants