Skip to content

Commit e03dc92

Browse files
committed
common: fix early stop of panic
Add a join() to the thread, otherwise it happens that the main process is already closed and the thread is still alive (because daemon = True). This causes issues in tests, cause the returncode is 0 instead of -9, as there was not enough time to execute the os.killpg before the main thread ended. Signed-off-by: Jean-Louis Dupond <jean-louis@dupond.be>
1 parent 67a29d0 commit e03dc92

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

lib/vdsm/common/panic.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,3 +23,4 @@ def run():
2323
logging.shutdown()
2424
finally:
2525
ready.set()
26+
t.join()

0 commit comments

Comments
 (0)