From: Zack Cerza <zack@cerza.org>
Date: Fri, 30 May 2014 14:56:27 +0000 (-0500)
Subject: Add comment explaining 'preexec_fn=os.setpgrp'
X-Git-Tag: 1.1.0~1426
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=0fcfaee726864c69350b357c5d362d1df46bc7e5;p=teuthology.git

Add comment explaining 'preexec_fn=os.setpgrp'

Signed-off-by: Zack Cerza <zack.cerza@inktank.com>
---

diff --git a/teuthology/worker.py b/teuthology/worker.py
index 7c80d9577..2e3891c44 100644
--- a/teuthology/worker.py
+++ b/teuthology/worker.py
@@ -235,6 +235,9 @@ def main(ctx):
                 '--name',
                 job_config['name'],
             ]
+            # Execute teuthology-results, passing 'preexec_fn=os.setpgrp' to
+            # make sure that it will continue to run if this worker process
+            # dies (e.g. because of a restart)
             result_pid = subprocess.Popen(args=args,
                                           preexec_fn=os.setpgrp,).pid
             log.info("teuthology-results PID: %s", result_pid)