Skip to content

Commit c9e4125

Browse files
committed
Fail build_report jobs on non-success statuses
1 parent eb23de3 commit c9e4125

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

tests/ci/build_report_check.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,8 @@ def main():
139139
additional_files=[report_path],
140140
).dump()
141141

142-
if summary_status == ERROR:
142+
# We should fail the report job to rerun it in the following attempts
143+
if summary_status != SUCCESS:
143144
sys.exit(1)
144145

145146

tests/ci/ci.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2121,8 +2121,6 @@ def main() -> int:
21212121
pr_info,
21222122
dump_to_file=True,
21232123
)
2124-
if job_report.status != SUCCESS:
2125-
exit_code = 1
21262124
if not pr_info.is_merge_queue:
21272125
# in the merge queue mergeable status must be set only in FinishCheck (last job in wf)
21282126
update_mergeable_check(

0 commit comments

Comments
 (0)