From d963fd596eaa2462dec55f354ed48f92010b0722 Mon Sep 17 00:00:00 2001 From: Sergi Blanch Torne Date: Mon, 27 May 2024 13:47:52 +0200 Subject: [PATCH] ci: fix stress counter in run'n'monitor The stress counter after enable_job(retry), often stores the new status instead of the job complete. So, the summary printed later doesn't show the real evolution of the test. Signed-off-by: Sergi Blanch Torne Part-of: --- bin/ci/ci_run_n_monitor.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/bin/ci/ci_run_n_monitor.py b/bin/ci/ci_run_n_monitor.py index 6ead6c6ec96..3a6af463d87 100755 --- a/bin/ci/ci_run_n_monitor.py +++ b/bin/ci/ci_run_n_monitor.py @@ -120,8 +120,8 @@ def monitor_pipeline( stress < 0 or sum(stress_status_counter[job.name].values()) < stress ): - job = enable_job(project, pipeline, job, "retry", force_manual) stress_status_counter[job.name][job.status] += 1 + job = enable_job(project, pipeline, job, "retry", force_manual) else: job = enable_job(project, pipeline, job, "target", force_manual) @@ -129,7 +129,7 @@ def monitor_pipeline( target_statuses[job.name] = job.status continue - # all jobs + # all other non-target jobs if job.status != statuses[job.name]: print_job_status(job, True) statuses[job.name] = job.status