Commit 70d2e4bc authored by Łukasz Nowak's avatar Łukasz Nowak

Setup process group for test suite.

Thanks to this registration to process_group_pid_set allows cleanup of test
suite too.

If something else is left behind, those are disconnected processes of test
suite itself, which shall be cleared in that one.
parent e94ff72d
...@@ -255,7 +255,8 @@ branch = %(branch)s ...@@ -255,7 +255,8 @@ branch = %(branch)s
# XXX: is it good for all cases (eg: test runner fails too early for # XXX: is it good for all cases (eg: test runner fails too early for
# any custom code to pick the failure up and react ?) # any custom code to pick the failure up and react ?)
remote_test_result_needs_cleanup = False remote_test_result_needs_cleanup = False
run_test_suite = subprocess.Popen(invocation_list) run_test_suite = subprocess.Popen(invocation_list,
preexec_fn=os.setsid)
process_group_pid_set.add(run_test_suite.pid) process_group_pid_set.add(run_test_suite.pid)
run_test_suite.wait() run_test_suite.wait()
process_group_pid_set.remove(run_test_suite.pid) process_group_pid_set.remove(run_test_suite.pid)
......
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