From: Zack Cerza <zack@cerza.org>
Date: Wed, 28 May 2014 19:56:59 +0000 (-0500)
Subject: Don't block on teuthology-results processes
X-Git-Tag: 1.1.0~1430
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=c4783543e3f80cb7b08d90c37388f302a04b0856;p=teuthology.git

Don't block on teuthology-results processes

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

diff --git a/teuthology/worker.py b/teuthology/worker.py
index 5a3f38c4b1..8746170f81 100644
--- a/teuthology/worker.py
+++ b/teuthology/worker.py
@@ -215,10 +215,10 @@ def main(ctx):
                                (teuthology_branch, teuth_bin_path))
 
         if job_config.get('last_in_suite'):
-            log.info('Generating results email for %s', job_config['name'])
             if teuth_config.results_server:
                 report.try_delete_jobs(job_config['name'],
                                        job_config['job_id'])
+            log.info('Generating results email for %s', job_config['name'])
             args = [
                 os.path.join(teuth_bin_path, 'teuthology-results'),
                 '--timeout',
@@ -230,7 +230,9 @@ def main(ctx):
                 '--name',
                 job_config['name'],
             ]
-            subprocess.Popen(args=args).wait()
+            result_pid = subprocess.Popen(args=args,
+                                          preexec_fn=os.setpgrp,).pid
+            log.debug("teuthology-results PID: %s", result_pid)
         else:
             log.info('Creating archive dir %s', archive_path_full)
             safepath.makedirs(ctx.archive_dir, safe_archive)