Commit 548a5325 authored by Kirill Smelkov's avatar Kirill Smelkov

Fix thinko when finally killing processes

First we send SIGTERM to leaked processes, and then, after timeout,
intend to send SIGKILL to the leaked processes and to the main test
process if it is still alive.

However there is a thiko in the code: we were sending SIGKILL only to
the main test process, not to all leaked ones.

-> Fix it.

The bug was there since 0ad45a9c (Detect if a test leaks processes and
terminate them).

/reviewed-by @jerome
/reviewed-on nexedi/nxdtest!16
parent b0cf277d
...@@ -348,7 +348,7 @@ def main(): ...@@ -348,7 +348,7 @@ def main():
proc.terminate() proc.terminate()
gone, alive = psutil.wait_procs(procv, timeout=5) gone, alive = psutil.wait_procs(procv, timeout=5)
for proc in alive: for proc in alive:
p.kill() proc.kill()
if err is not None: if err is not None:
raise err raise err
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment