hundreds-breakfast-49010
09/23/2020, 12:17 AM./pants repl
, then hit ctrl-c, some kind of python process will remain around and screwing up the terminal/home/gregs/code/pants/build-support/virtualenvs/Linux/pants_dev_deps.py38.venv/bin/python3 /home/gregs/code/pants/.pants.d/tmp0rx6pokk/requirements.pex
, which is I think the actual process that runs when pants creates a python repl. I'm not sure why this is getting killed when the pytest process is getting killed (with the above commit)witty-crayon-22786
09/23/2020, 12:31 AMhundreds-breakfast-49010
09/23/2020, 12:33 AMhundreds-father-404
09/23/2020, 12:45 AMwitty-crayon-22786
09/23/2020, 12:46 AMaloof-angle-91616
09/23/2020, 12:47 AMwitty-crayon-22786
09/23/2020, 12:52 AMhundreds-breakfast-49010
09/23/2020, 12:53 AMwitty-crayon-22786
09/23/2020, 12:53 AMrun
and repl
…hundreds-breakfast-49010
09/23/2020, 12:53 AMaloof-angle-91616
09/23/2020, 12:53 AMhundreds-breakfast-49010
09/23/2020, 12:54 AMaloof-angle-91616
09/23/2020, 12:55 AMhundreds-breakfast-49010
09/23/2020, 12:55 AMchildren
thing psutil
gives youaloof-angle-91616
09/23/2020, 12:56 AMwitty-crayon-22786
09/23/2020, 12:58 AMaloof-angle-91616
09/23/2020, 12:59 AMhundreds-breakfast-49010
09/23/2020, 12:59 AMwitty-crayon-22786
09/23/2020, 1:00 AMthat’s going to be challenging even/especially with the heartbeat strategy.to clarify: it’s easy to not kill pantsd with the heartbeat strategy. but it’s challenging for the server to tear down processes cleanly
aloof-angle-91616
09/23/2020, 1:00 AMwitty-crayon-22786
09/23/2020, 1:01 AMpantsd
should stay up, but the run associated with the client should definitely diealoof-angle-91616
09/23/2020, 1:04 AMwitty-crayon-22786
09/23/2020, 1:06 AMaloof-angle-91616
09/23/2020, 1:08 AMwitty-crayon-22786
09/23/2020, 1:12 AMhundreds-breakfast-49010
09/23/2020, 1:15 AMwitty-crayon-22786
09/23/2020, 1:15 AMaloof-angle-91616
09/23/2020, 1:16 AMInteractiveProcess
and ^C
. i hope we don't have to use either of them https://github.com/pantsbuild/pants/issues/10057#issuecomment-698845410