[WIP] bpo-38546: Add support.get_child_processes() by vstinner · Pull Request #17640 · python/cpython

the-knights-who-say-ni

Conversation

libregrtest runtest() now checks if the test leaks direct child
process running in the background.

Choose a reason for hiding this comment

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

LGTM! One key point is that it fails gracefully on systems without /proc

I failed to prove that this PR helps to detect leaked processes. I prefer to not merge it until I managed to prove that it's useful. In the meanwhile, there are already existing checks which should detect some cases.

See https://pythondev.readthedocs.io/unstable_tests.html for my notes on unstable tests.

[WIP] bpo-38546: Add support.get_child_processes() by vstinner · Pull Request #17640 · python/cpython

Conversation

Choose a reason for hiding this comment

Labels