From 03b8cdac387249f12dcbf92bf01b9678426d66a8 Mon Sep 17 00:00:00 2001 From: Zack Cerza Date: Thu, 17 Apr 2014 11:17:22 -0500 Subject: [PATCH] Refactor try_delete_jobs() Also tweak its error message --- teuthology/report.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/teuthology/report.py b/teuthology/report.py index de1f5e42..c8c290b8 100644 --- a/teuthology/report.py +++ b/teuthology/report.py @@ -436,12 +436,15 @@ def try_delete_jobs(run_name, job_ids): reporter = ResultsReporter() log.debug("Deleting jobs from {server}: {jobs}".format( server=config.results_server, jobs=str(job_ids))) - for job_id in job_ids: + + def try_delete_job(job_id): with safe_while(_raise=False) as proceed: while proceed(): try: reporter.delete_job(run_name, job_id) return except (requests.exceptions.RequestException, socket.error): - log.exception("Could not report results to %s", - config.results_server) + log.exception("Job deletion failed") + + for job_id in job_ids: + try_delete_job(job_id) -- 2.47.3