From: Zack Cerza <zack@cerza.org>
Date: Mon, 4 Nov 2013 20:56:13 +0000 (-0600)
Subject: Don't use create_run() unless necessary
X-Git-Tag: v0.94.10~27^2^2~364^2~491^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=ed81960242ca105a09beadf0b062201318ee6823;p=ceph.git

Don't use create_run() unless necessary

Runs are created automatically now.

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

diff --git a/teuthology/report.py b/teuthology/report.py
index 9d53050d03e..a67a1dccaba 100644
--- a/teuthology/report.py
+++ b/teuthology/report.py
@@ -255,14 +255,12 @@ class ResultsReporter(object):
             jobs=len(jobs),
         ))
         if jobs:
-            status, msg, content = self.create_run(run_name)
-            if status == 200:
-                self.report_jobs(run_name, jobs.keys())
-            elif msg.endswith('already exists'):
-                if self.refresh:
-                    self.report_jobs(run_name, jobs.keys())
-                else:
+            if not self.refresh:
+                status, msg, content = self.create_run(run_name)
+                if status != 200:
                     log.info("    already present; skipped")
+                    return
+            self.report_jobs(run_name, jobs.keys())
         elif not jobs:
             log.debug("    no jobs; skipped")
         return len(jobs)
@@ -392,7 +390,6 @@ def try_push_job_info(job_config, extra_info=None):
 
     try:
         log.info("Pushing job info to %s", config.results_server)
-        create_run(run_name)
         push_job_info(run_name, job_id, job_info)
     except RequestFailedError:
         log.exception("Could not report results to %s" %