You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iggy.apache.org by hg...@apache.org on 2026/10/16 16:20:40 UTC
(iggy) 03/03: fix(ci): keep job timing API errors from failing finalize_pr
This is an automated email from the ASF dual-hosted git repository.
hubcio pushed a commit to branch fix/remove-cg-session-timeout
in repository https://gitbox.apache.org/repos/asf/iggy.git
commit 27f6a7660f02ac098a2080d92412321e83947288
Author: Hubert Gruszecki <h....@gmail.com>
AuthorDate: Fri Oct 16 18:19:11 2026 +0200
fix(ci): keep job timing API errors from failing finalize_pr
On re-run attempts of full-matrix runs, the job listing in "Get job
execution times" exceeds the ~10 s GitHub gateway timeout and gets
a 502. The unhandled error failed the step, so "Check status" never
ran and the required finalize_pr check failed with all jobs green.
The durations only fill the summary table, and the gate reads
needs.*.result, so the step can now fail without failing the job.
---
.github/workflows/pre-merge.yml | 3 +++
1 file changed, 3 insertions(+)
diff --git a/.github/workflows/pre-merge.yml b/.github/workflows/pre-merge.yml
index 13456e635..956ca8384 100644
--- a/.github/workflows/pre-merge.yml
+++ b/.github/workflows/pre-merge.yml
@@ -213,8 +213,11 @@ jobs:
needs: [common, detect, runtime, rust-test-build, test-rust, test-rust-runtime, test-python, test-php, test-node, test-go, test-java, test-csharp, test-cpp, test-swift, test-bdd, test-examples, test-other]
if: ${{ !cancelled() }}
steps:
+ # The durations only fill the summary table. A GitHub API error here
+ # must not skip "Check status", which decides the required check.
- name: Get job execution times
id: times
+ continue-on-error: true
uses: actions/github-script@v9
with:
script: |