From 6c19a067a0b9a6d725a2f94022f2f01ca4b10d76 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Luk=C3=A1=C5=A1=20Doktor?= Date: Fri, 22 Nov 2024 09:11:31 +0100 Subject: [PATCH] ci.gatekeeper: Update existing results MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit tha matching run_id means we're dealing with the same job but with updated results and not with an older job. Update the results in such case. Signed-off-by: Lukáš Doktor --- tools/testing/gatekeeper/jobs.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/tools/testing/gatekeeper/jobs.py b/tools/testing/gatekeeper/jobs.py index e6cc120801..cbf2b5a8f4 100644 --- a/tools/testing/gatekeeper/jobs.py +++ b/tools/testing/gatekeeper/jobs.py @@ -80,9 +80,7 @@ class Checker: else: # Not a required job return - # TODO: Check if multiple re-runs use the same "run_id". If so use - # job['run_attempt'] in case of matching "run_id". - elif job['run_id'] <= self.results[job_name]['run_id']: + elif job['run_id'] < self.results[job_name]['run_id']: # Newer results already stored print(f"older {job_name} - {job['status']} {job['conclusion']} " f"{job['id']}", file=sys.stderr)