You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xtable.apache.org by "rangareddy (via GitHub)" <gi...@apache.org> on 2026/07/24 03:54:00 UTC
[PR] ci: add aggregate gate job for spark-runtime matrix (single required check) [incubator-xtable]
rangareddy opened a new pull request, #862:
URL: https://github.com/apache/incubator-xtable/pull/862
## What
Adds a single aggregate "gate" job to `spark-runtime-validation.yml` so branch protection can require **one stable status context** for the Spark-runtime bundle validation instead of listing every matrix version.
Follow-up to the review discussion on #848: `.asf.yaml` / GitHub branch protection match required status checks by exact name (no wildcards), so a matrix job forces you to enumerate `validate-spark-runtime-bundle (3.4.3)` and `validate-spark-runtime-bundle (3.5.9)` — and re-edit `.asf.yaml` every time the Spark matrix changes.
## Change
- New job **`spark-runtime-validation-required`** that `needs: validate-spark-runtime-bundle` and fails unless the aggregated matrix result is `success`. `needs.<job>.result` is `success` only when all matrix legs pass.
- `if: always()` is used deliberately: without it the gate is *skipped* when a leg fails, and a skipped required check leaves a PR stuck on a pending status.
The matrix job is unchanged.
## Follow-up on `main` (#848)
Once this lands on `branch-0.4`, `#848` can require just the one context for `branch-0.4`:
```yaml
branch-0.4:
required_status_checks:
contexts:
- "Maven CI Build"
- "License Check"
- "spark-runtime-validation-required"
```
(dropping the two versioned `validate-spark-runtime-bundle (…)` entries). I've left the same suggestion on the #848 review thread. Merge order: this PR first, then update `.asf.yaml` in #848.
## Testing
YAML validated locally; job graph confirmed (`spark-runtime-validation-required` depends on the matrix job, `if: always()`, exits non-zero unless the aggregated result is `success`).
--
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: commits-unsubscribe@xtable.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] ci: aggregate gate job + branch-0.4 required checks for spark-runtime [incubator-xtable]
Posted by "rangareddy (via GitHub)" <gi...@apache.org>.
rangareddy commented on PR #862:
URL: https://github.com/apache/incubator-xtable/pull/862#issuecomment-5091484773
CC @vinishjail97 @the-other-tim-brown
--
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: commits-unsubscribe@xtable.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org