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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
