Skip to content

ci(e2e): run partition check parallel to test jobs#825

Merged
vieiralucas merged 1 commit intomainfrom
ci-decouple-e2e-partition-check
Apr 28, 2026
Merged

ci(e2e): run partition check parallel to test jobs#825
vieiralucas merged 1 commit intomainfrom
ci-decouple-e2e-partition-check

Conversation

@vieiralucas
Copy link
Copy Markdown
Member

@vieiralucas vieiralucas commented Apr 28, 2026

Summary

  • e2e-partition-check takes ~10m and was gating every e2e matrix job via needs:
  • It still runs as a top-level job and still fails the workflow on drift, so gating just adds wall time without adding safety
  • Drop it from e2e.needs: so partition jobs start as soon as e2e-build finishes

Test plan

  • CI: e2e-partition-check and e2e (*) jobs start in parallel
  • Workflow still fails red if partition coverage drift introduced

Summary by cubic

Run e2e-partition-check in parallel with the e2e matrix jobs by removing it from e2e.needs, so partitions start right after e2e-build. This reduces wall time without reducing safety, since the partition check still runs as a top-level job and fails the workflow on drift.

Written for commit ac09b0a. Summary will update on new commits. Review in cubic

e2e-partition-check takes ~10m and was gating all e2e partition jobs
via needs:. Drift still fails the workflow via the separate job, so
gating buys nothing. Drop it from the e2e job's needs: list so test
partitions start as soon as e2e-build finishes.
@vieiralucas vieiralucas merged commit 52c7467 into main Apr 28, 2026
9 checks passed
@vieiralucas vieiralucas deleted the ci-decouple-e2e-partition-check branch April 28, 2026 18:54
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 28, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant