[issue41739] test_logging: test_race_between_set_target_and_flush() leaks threads (threading_cleanup() failed to cleanup ...)

2020-10-12 Thread STINNER Victor
STINNER Victor added the comment: Ok, test_race_between_set_target_and_flush() should no longer leak dangling threads. I close the issue. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue41739] test_logging: test_race_between_set_target_and_flush() leaks threads (threading_cleanup() failed to cleanup ...)

2020-10-12 Thread STINNER Victor
STINNER Victor added the comment: New changeset 33057c70920787627f2c94d08eb5d33c31f8bdd9 by Miss Skeleton (bot) in branch '3.8': bpo-41739: Fix test_logging.test_race_between_set_target_and_flush() (GH-22655) (GH-22656) (GH-22662)

[issue41739] test_logging: test_race_between_set_target_and_flush() leaks threads (threading_cleanup() failed to cleanup ...)

2020-10-11 Thread Vinay Sajip
Vinay Sajip added the comment: New changeset f5393dc2a0ced7bf670ebc56b5fd10a3eb328d1a by Victor Stinner in branch '3.9': bpo-41739: Fix test_logging.test_race_between_set_target_and_flush() (GH-22655) (GH-22656)

[issue41739] test_logging: test_race_between_set_target_and_flush() leaks threads (threading_cleanup() failed to cleanup ...)

2020-10-11 Thread miss-islington
Change by miss-islington : -- nosy: +miss-islington nosy_count: 4.0 -> 5.0 pull_requests: +21638 pull_request: https://github.com/python/cpython/pull/22662 ___ Python tracker

[issue41739] test_logging: test_race_between_set_target_and_flush() leaks threads (threading_cleanup() failed to cleanup ...)

2020-10-11 Thread STINNER Victor
Change by STINNER Victor : -- pull_requests: +21632 pull_request: https://github.com/python/cpython/pull/22656 ___ Python tracker ___

[issue41739] test_logging: test_race_between_set_target_and_flush() leaks threads (threading_cleanup() failed to cleanup ...)

2020-10-11 Thread STINNER Victor
STINNER Victor added the comment: New changeset 13ff396c019d548ba181cf22c6f39309a300723c by Victor Stinner in branch 'master': bpo-41739: Fix test_logging.test_race_between_set_target_and_flush() (GH-22655) https://github.com/python/cpython/commit/13ff396c019d548ba181cf22c6f39309a300723c

[issue41739] test_logging: test_race_between_set_target_and_flush() leaks threads (threading_cleanup() failed to cleanup ...)

2020-10-11 Thread STINNER Victor
Change by STINNER Victor : -- keywords: +patch pull_requests: +21631 stage: -> patch review pull_request: https://github.com/python/cpython/pull/22655 ___ Python tracker ___

[issue41739] test_logging: test_race_between_set_target_and_flush() leaks threads (threading_cleanup() failed to cleanup ...)

2020-10-11 Thread STINNER Victor
STINNER Victor added the comment: I'm able to reproduce the issue if I reduce the sleep: diff --git a/Lib/test/test_logging.py b/Lib/test/test_logging.py index 4cd8c7e25d..fc395dc473 100644 --- a/Lib/test/test_logging.py +++ b/Lib/test/test_logging.py @@ -1177,7 +1177,7 @@ class

[issue41739] test_logging: test_race_between_set_target_and_flush() leaks threads (threading_cleanup() failed to cleanup ...)

2020-10-11 Thread STINNER Victor
STINNER Victor added the comment: Cool, my threading change works as expected! The name of the target function was logged. https://buildbot.python.org/all/#/builders/509/builds/172 0:00:28 load avg: 3.58 [ 36/424/1] test_logging failed (env changed) Warning -- threading_cleanup() failed to