nevzheng commented on PR #12545: URL: https://github.com/apache/gravitino/pull/12545#issuecomment-5406233417
@jerryshao ledger after a wider search, then the three review fixes. **Why other CI names disappeared:** the children no longer self-listen on `pull_request`. They run as nested jobs under **Required CI**. On this PR you should still see one check per suite, nested under the parent — not 14 independent workflow runs. That is the intended shape (Marc Philipp, 2026-08-10: one required check; path filters inside jobs). **What was actually broken (your review, 2026-08-21):** | ID | Ask | Disposition | | --- | --- | --- | | J1 | Drop unrelated SPIP | Already dropped; threads left open | | J2 | Shared `'standalone'` cancels push CI | Fixed in 5b48b8af1: `required-<suite>` \|\| `github.workflow` | | J3 | `web-ui-tests` lost `web/web/**` | Fixed in 5b48b8af1: internal `changes` / dorny | | J4 | Parent `cancel-in-progress: false` serializes PRs | Fixed in 5b48b8af1: `true` | | J5 | Other CI doesn't run | Children nest under Required CI; after J2, push-to-main should stop eating sibling suites | **Internet notes we used:** `github.event_name` inside a reusable workflow is the caller's event, never `workflow_call` (actions/runner#3146). `github.workflow` in a callee is the caller name — using it on both sides deadlocks (SO 78101326). Required path stays on unique `required-*` literals for that reason. Source contract: `python3 dev/ci/test_required_ci.py` (passed locally; does not start Actions). Nevin Sent from my 🤖 (Cursor) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
