(texera) branch release/v1.1.0-incubating updated: fix(ci): treat skipped backport jobs as green in Direct Backport Push (#4629)

[github-bot]

This is an automated email from the ASF dual-hosted git repository.

github-actions[bot] pushed a commit to branch release/v1.1.0-incubating
in repository https://gitbox.apache.org/repos/asf/texera.git


The following commit(s) were added to refs/heads/release/v1.1.0-incubating by 
this push:
     new 80a487232d fix(ci): treat skipped backport jobs as green in Direct 
Backport Push (#4629)
80a487232d is described below

commit 80a487232dfa2079eb2dbf294c8074edbd9b8e96
Author: Yicong Huang <17627829+yicong-hu...@users.noreply.github.com>
AuthorDate: Sat May 2 00:41:08 2026 +0000

    fix(ci): treat skipped backport jobs as green in Direct Backport Push 
(#4629)
    
    ### What changes were proposed in this PR?
    
    After #4622 the `precheck` step legitimately skips build stacks based on
    PR labels (e.g., a PR without the `frontend` label skips the frontend
    stack). The backport caller propagates these toggles, so the
    corresponding `backport (target) / frontend (...)` job ends with
    `conclusion: "skipped"`.
    
    `direct-backport-push.yml`'s discovery filter required **every**
    matching job to be `success`, so legitimately-skipped jobs caused the
    whole target to be classified as not-green and `push-backports` was
    silently skipped. Match the `Required Checks` aggregator's logic by
    accepting both `success` and `skipped`.
    
    ### Any related issues, documentation, discussions?
    
    Closes #4628. Triggered by run
    [25237976528](https://github.com/apache/texera/actions/runs/25237976528)
    for the merge of #4622.
    
    ### How was this PR tested?
    
    Single-line filter change; YAML parses locally. Will be exercised on the
    next merge of a PR with `release/*` labels and a partially-skipped build
    matrix.
    
    ### Was this PR authored or co-authored using generative AI tooling?
    
    Generated-by: Claude Code (Opus 4.7)
    
    Co-authored-by: Claude Opus 4.7 (1M context) <nore...@anthropic.com>
    
    (backported from commit 8e40078e2fe2f9b51366bbcec511ba4a667b37ce)
---
 .github/workflows/direct-backport-push.yml | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/direct-backport-push.yml 
b/.github/workflows/direct-backport-push.yml
index c291fc1887..4ec6483645 100644
--- a/.github/workflows/direct-backport-push.yml
+++ b/.github/workflows/direct-backport-push.yml
@@ -156,7 +156,14 @@ jobs:
               greenTargets = requestedTargets.filter((target) => {
                 const prefix = `backport (${target}) / `;
                 const targetJobs = allJobs.filter((job) => 
job.name.startsWith(prefix));
-                return targetJobs.length > 0 && targetJobs.every((job) => 
job.conclusion === "success");
+                // Treat skipped as green: precheck legitimately skips stacks
+                // based on PR labels, matching the Required Checks aggregator.
+                return (
+                  targetJobs.length > 0 &&
+                  targetJobs.every(
+                    (job) => job.conclusion === "success" || job.conclusion 
=== "skipped"
+                  )
+                );
               });
             }