From ed81960242ca105a09beadf0b062201318ee6823 Mon Sep 17 00:00:00 2001 From: Zack Cerza Date: Mon, 4 Nov 2013 14:56:13 -0600 Subject: [PATCH] Don't use create_run() unless necessary Runs are created automatically now. Signed-off-by: Zack Cerza --- teuthology/report.py | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/teuthology/report.py b/teuthology/report.py index 9d53050d03..a67a1dccab 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" % -- 2.39.5