Skip to content

test_asyncio leaks references after commit 9d2e1ea3862e5950d48b45ac57995a206e33f38b #129158

Closed
@pablogsal

Description

@pablogsal

Bug report

Bug description:

Executing

./python -m test test_asyncio.test_subprocess test_asyncio.test_waitfor -R 3:

leaks references on main. Bisecting points to

9d2e1ea3862e5950d48b45ac57995a206e33f38b is the first bad commit
commit 9d2e1ea3862e5950d48b45ac57995a206e33f38b
Author: Kumar Aditya <kumaraditya@python.org>
Date:   Sun Jun 23 18:38:50 2024 +0530

    GH-120804: Remove `PidfdChildWatcher`, `ThreadedChildWatcher` and `AbstractChildWatcher` from asyncio APIs (#120893)

 Lib/asyncio/unix_events.py                | 199 ++++--------------------------
 Lib/test/test_asyncio/test_events.py      |  26 +---
 Lib/test/test_asyncio/test_subprocess.py  |  30 ++---
 Lib/test/test_asyncio/test_unix_events.py |  26 ----
 Lib/test/test_asyncio/utils.py            |  17 ---
 5 files changed, 39 insertions(+), 259 deletions(-)
bisect found first bad commit

Unclear why the buildbots don't catch this but I am catching it everytime I check test_asyncio for refleaks.

CC: @kumaraditya303

CPython versions tested on:

CPython main branch

Operating systems tested on:

Linux

Linked PRs

Metadata

Metadata

Labels

testsTests in the Lib/test dirtopic-asynciotype-bugAn unexpected behavior, bug, or error

Projects

Status

Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions