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 <sergi.blanch.torne@collabora.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/29416>
This commit is contained in:

committed by
Marge Bot

parent
a47c5c9eee
commit
d963fd596e
@@ -120,8 +120,8 @@ def monitor_pipeline(
|
|||||||
stress < 0
|
stress < 0
|
||||||
or sum(stress_status_counter[job.name].values()) < stress
|
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
|
stress_status_counter[job.name][job.status] += 1
|
||||||
|
job = enable_job(project, pipeline, job, "retry", force_manual)
|
||||||
else:
|
else:
|
||||||
job = enable_job(project, pipeline, job, "target", force_manual)
|
job = enable_job(project, pipeline, job, "target", force_manual)
|
||||||
|
|
||||||
@@ -129,7 +129,7 @@ def monitor_pipeline(
|
|||||||
target_statuses[job.name] = job.status
|
target_statuses[job.name] = job.status
|
||||||
continue
|
continue
|
||||||
|
|
||||||
# all jobs
|
# all other non-target jobs
|
||||||
if job.status != statuses[job.name]:
|
if job.status != statuses[job.name]:
|
||||||
print_job_status(job, True)
|
print_job_status(job, True)
|
||||||
statuses[job.name] = job.status
|
statuses[job.name] = job.status
|
||||||
|
Reference in New Issue
Block a user