[cadence] Run build/test on fork exports and surface per-job checks on the PR#20374
[cadence] Run build/test on fork exports and surface per-job checks on the PR#20374aliafzal wants to merge 2 commits into
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/20374
Note: Links to docs will display an error until the docs builds have been completed. ❌ 4 New Failures, 2 Unrelated FailuresAs of commit 673fc64 with merge base 24d8bab ( NEW FAILURES - The following jobs have failed:
FLAKY - The following job failed but was likely due to flakiness present on trunk:
BROKEN TRUNK - The following job failed but was present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This PR needs a
|
…n the PR The Cadence Build & Test workflow ran its credentialed build/test jobs only on same-repo PRs. On Meta-exported fork PRs (labeled meta-exported + CLA Signed) the cpu-build and cpu-test jobs were excluded from the fork (pull_request_target) path, and the jobs that did run there did not appear as checks on the PR because pull_request_target check runs attach to the base SHA. Each cadence job now carries an inlined run condition: push/schedule/workflow_dispatch, same-repo pull_request, or a fork pull_request_target labeled CLA Signed + meta-exported. There is no separate gate job; GitHub Actions has no YAML anchors and env is unavailable in job-level if, so the condition is duplicated per job. export-checks-start/export-checks-finish mirror per-job check runs onto the PR head SHA so each cadence job shows up as its own check on the export PR, linking to its logs. Same-repo PRs continue to run via pull_request and report checks natively. Authored with Claude Code.
fffbfa4 to
481dcac
Compare
0595ba8 to
481dcac
Compare
…Checks API only (will revert)
|
Superseded by #20388 (all cadence jobs moved to pull_request_target). |
Summary
The Cadence Build & Test workflow ran its credentialed build/test jobs only on same-repo PRs. On Meta-exported fork PRs (labeled
meta-exported+CLA Signed) thecpu-buildandcpu-testjobs were excluded from thepull_request_targetpath, and the jobs that did run there did not appear as checks on the PR becausepull_request_targetcheck runs attach to the base SHA.This change:
cpu-buildandcpu-testthrough the samegateashifi-build/vision-build, so all four credentialed jobs run on fork exports (and skip cleanly on non-exported fork PRs). Their checkoutrefnow also honorspull_request_target.export-checks-start/export-checks-finish, which mirror per-job check runs onto the PR head SHA via the Checks API, so each cadence job shows up as its own check on the export PR (in-progress then success/failure/skipped), each linking to its job logs.Same-repo PRs continue to run via
pull_requestand report their checks natively; the new mirror jobs only run on thepull_request_targetpath.Test plan
pull_requestrunsgate+cpu-build/cpu-test/hifi-build/vision-buildwith native checks.meta-exported+CLA Signedare applied, thepull_request_targetrun executes the four jobs with credentials and postscadence (export) / <job>checks onto the PR head.Authored with Claude Code.