[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Eric Snow added the comment: @Pablo, yeah, I'll try to take a look this week. Are these new failures? Keep in mind that these tests can sometimes expose existing bugs in our runtime (as we fix runtime issues elsewhere) rather than regressions, though that isn't necessarily the case here. :)

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Eric Snow added the comment: New changeset 818f5b597ae93411cc44e404544247d436026a00 by Eric Snow in branch 'master': bpo-32604: Clean up test.support.interpreters. (gh-20926) https://github.com/python/cpython/commit/818f5b597ae93411cc44e404544247d436026a00 -- __

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Change by Eric Snow : -- pull_requests: +20105 pull_request: https://github.com/python/cpython/pull/20926 ___ Python tracker ___ ___

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Change by Joannah Nanjekye : -- pull_requests: +19974 pull_request: https://github.com/python/cpython/pull/20777 ___ Python tracker ___

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Pablo Galindo Salgado added the comment: Joannah, Eric, could you look into this? Having random failures in the builbots can make debugging more challenging, hide other issues and can also spam the buildbot list :( -- ___ Python tracker

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Change by Ned Deily : -- nosy: -ned.deily ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pytho

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

STINNER Victor added the comment: test__xxsubinterpreters and test_interpreters failed on AMD64 FreeBSD Shared 3.x: https://buildbot.python.org/all/#/builders/152/builds/973 See also bpo-37224: "[subinterpreters] test__xxsubinterpreters fails randomly". IMO there are multiple race conditions

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Joannah Nanjekye added the comment: New changeset bae872f1fe9b3a0d3e3b8800a2ac8d6b440d6e4d by Joannah Nanjekye in branch 'master': bpo-32604: Recommit "bpo-32604: PEP 554 for use in test suite (GH-19985)" (GH-20611) https://github.com/python/cpython/commit/bae872f1fe9b3a0d3e3b8800a2ac8d6b440

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Change by Joannah Nanjekye : -- pull_requests: +19839 pull_request: https://github.com/python/cpython/pull/20611 ___ Python tracker ___

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

STINNER Victor added the comment: New changeset 18a90248fdd92b27098cc4db773686a2d10a4d24 by Victor Stinner in branch 'master': bpo-32604: Fix reference leak in select module (GH-20600) https://github.com/python/cpython/commit/18a90248fdd92b27098cc4db773686a2d10a4d24 --

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Change by STINNER Victor : -- pull_requests: +19829 pull_request: https://github.com/python/cpython/pull/20600 ___ Python tracker ___ __

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

STINNER Victor added the comment: The following test is enough to reproduce the leak: --- import unittest import _xxsubinterpreters as _interpreters class TestInterpreterDestroy(unittest.TestCase): def tearDown(self): for interp_id in _interpreters.list_all(): if inter

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

STINNER Victor added the comment: New changeset 7d80b35af1ee03834ae4af83e920dee89c2bc273 by Pablo Galindo in branch 'master': Revert "bpo-32604: PEP 554 for use in test suite (GH-19985)" (#20465) https://github.com/python/cpython/commit/7d80b35af1ee03834ae4af83e920dee89c2bc273 -- _

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Pablo Galindo Salgado added the comment: Commit 9d17cbf33df7cfb67ca0f37f6463ba5c18676641 is causing all refleak buildbots to fail and is masking other issues and causing some confusion already in other PRs (like https://github.com/python/cpython/pull/20433) so I opened a revert in PR 20465 f

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Change by Pablo Galindo Salgado : -- nosy: +pablogsal nosy_count: 13.0 -> 14.0 pull_requests: +19717 pull_request: https://github.com/python/cpython/pull/20465 ___ Python tracker _

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Joannah Nanjekye added the comment: I will look at this tommorrow. Am abit busy today. Thanks Victor -- ___ Python tracker ___ ___

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

STINNER Victor added the comment: The new test leaks references: https://buildbot.python.org/all/#/builders/563/builds/105 test_interpreters leaked [216, 216, 216] references, sum=648 test_interpreters leaked [84, 84, 84] memory blocks, sum=252 Use "./python -m test.bisect_cmd -R 3:3 test_in

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Joannah Nanjekye added the comment: New changeset 9d17cbf33df7cfb67ca0f37f6463ba5c18676641 by Joannah Nanjekye in branch 'master': bpo-32604: PEP 554 for use in test suite (GH-19985) https://github.com/python/cpython/commit/9d17cbf33df7cfb67ca0f37f6463ba5c18676641 -- __

[issue32604] [subinterpreters] PEP 554 implementation: add interpreters module

Change by STINNER Victor : -- title: [subinterpreters] Expose the subinterpreters C-API in Python for testing use. -> [subinterpreters] PEP 554 implementation: add interpreters module ___ Python tracker