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: |