Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 34k
gh-128770: raise warnings as errors in test suite - except for test_socket which still logs warnings, and internal test warnings that are now logged#128973
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
Uh oh!
There was an error while loading. Please reload this page.
Conversation
…cept for test_socket which still logs warnings (python#128726)" (python#128936) This reverts commit 76856ae.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
graingert commented Jan 18, 2025
@hugovk can I get a run on the buildbots for this PR? |
hugovk commented Jan 18, 2025
Certainly! Let's check if you can do it yourself: either apply one of the |
bedevere-bot commented Jan 18, 2025
🤖 New build scheduled with the buildbot fleet by @graingert for commit c06c3db 🤖 If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again. |
bedevere-bot commented Jan 18, 2025
🤖 New build scheduled with the buildbot fleet by @graingert for commit 7123093 🤖 If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again. |
graingert commented Jan 18, 2025
@hugovk do you know where I can see the status? |
hugovk commented Jan 18, 2025
graingert commented Jan 18, 2025
hugovk commented Mar 27, 2025
The warnings have been fixed in other PRs in |
8a00c9a into python:mainUh oh!
There was an error while loading. Please reload this page.
Thanks @graingert for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13. |
Sorry, @graingert and @hugovk, I could not cleanly backport this to |
Sorry, @graingert and @hugovk, I could not cleanly backport this to |
GH-131802 is a backport of this pull request to the 3.13 branch. |
…test_socket which still logs warnings, and internal test warnings that are now logged (python#128973) Co-authored-by: Bénédikt Tran <[email protected]> (cherry picked from commit 8a00c9a)
bedevere-bot commented Mar 27, 2025
|
graingert commented Mar 27, 2025
the failure is related to this commit, however the code in this PR is working as intended. This has uncovered an issue in subinterpreters: test_create_many_threaded (test.test_interpreters.test_stress.StressTests.test_create_many_threaded) ... Warning--Uncaughtthreadexception: InterpreterErrorExceptioninthreadThread-166 (task): RuntimeError: FailedtoimportencodingsmoduleDuringhandlingoftheaboveexception, anotherexceptionoccurred: interpreters.InterpreterError: sub-interpretercreationfailedDuringhandlingoftheaboveexception, anotherexceptionoccurred: Traceback (mostrecentcalllast): File"/home/buildbot/buildarea/3.x.cstratak-fedora-stable-aarch64.lto/build/Lib/threading.py", line1054, in_bootstrap_innerself.run() ~~~~~~~~^^File"/home/buildbot/buildarea/3.x.cstratak-fedora-stable-aarch64.lto/build/Lib/threading.py", line996, inrunself._target(*self._args, **self._kwargs) ~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^File"/home/buildbot/buildarea/3.x.cstratak-fedora-stable-aarch64.lto/build/Lib/test/test_interpreters/test_stress.py", line30, intaskinterp=interpreters.create() File"/home/buildbot/buildarea/3.x.cstratak-fedora-stable-aarch64.lto/build/Lib/test/support/interpreters/__init__.py", line76, increateid=_interpreters.create(reqrefs=True) interpreters.InterpreterError: interpretercreationfailed |
ZeroIntensity commented Mar 27, 2025
That's a known failure, don't worry about it. |
graingert commented Mar 27, 2025
Ah you're right it's not a Warning that got raised as an Exception! I panicked like the message said not to |
GH-131824 is a backport of this pull request to the 3.12 branch. |
…test_socket which still logs warnings, and internal test warnings that are now logged (python#128973) Co-authored-by: Bénédikt Tran <[email protected]>




Uh oh!
There was an error while loading. Please reload this page.