Skip to content

Build the Linux wheels on OSDC runners - #9668

Open
huydhn wants to merge 4 commits into
mainfrom
osdc/wheels-on-osdc
Open

Build the Linux wheels on OSDC runners#9668
huydhn wants to merge 4 commits into
mainfrom
osdc/wheels-on-osdc

Conversation

@huydhn

@huydhn huydhn commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Second of three taking vision's Linux CI off EC2, after #9667.

One input on the matrix generator. pytorch/test-infra#8789 adds runner-fleet, which maps each row's validation_runner to its ARC equivalent — release-isolated on the refs these workflows publish from, regular otherwise, so PR builds move off EC2 too.

arch rows regular release
x86_64 cpu / rocm / xpu mt-l-x86iavx512-8-64 mt-rel-l-x86iavx512-44-340
x86_64 cuda mt-l-x86aavx2-29-113-a10g mt-rel-l-x86aavx2-29-113-l4
aarch64 any mt-l-arm64g4-16-62 mt-rel-l-arm64g3-44-340

The per-row split matters for x86: the matrix mixes four CPU rows with three cuda ones, and test/smoke_test.py guards its CUDA checks with torch.cuda.is_available(), so a cuda row on a CPU runner would skip them rather than fail. aarch64 gets one label either way, matching the runner it replaces — linux.arm64.m7g.4xlarge has no device either.

Depends on pytorch/test-infra#8789.

Authored with Claude Code.

@pytorch-bot

pytorch-bot Bot commented Sep 12, 2026

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/vision/9668

Note: Links to docs will display an error until the docs builds have been completed.

❌ 3 Pending, 1 Unclassified Failure

As of commit 9b506c5 with merge base 447c937 (image):

UNCLASSIFIED FAILURE - DrCI could not classify the following job because the workflow did not run on the merge base. The failure may be pre-existing on trunk or introduced by this PR:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

One input on the matrix generator. pytorch/test-infra#8789 adds runner-fleet,
which maps each row's validation_runner to its ARC equivalent: release-isolated
on the refs these workflows publish from (nightly, main, release/* and the v*
candidate tags), regular otherwise, so pull request builds move off EC2 too.

    x86_64  cpu/rocm/xpu  mt-l-x86iavx512-8-64      mt-rel-l-x86iavx512-44-340
    x86_64  cuda          mt-l-x86aavx2-29-113-a10g mt-rel-l-x86aavx2-29-113-l4
    aarch64 any           mt-l-arm64g4-16-62        mt-rel-l-arm64g3-44-340

The split matters for x86: the matrix mixes four CPU rows (cpu, rocm10_0,
rocm7_14, xpu) with three cuda ones, and test/smoke_test.py guards its CUDA
checks with torch.cuda.is_available(), so a cuda row on a CPU runner would skip
them rather than fail. aarch64 gets one label either way, matching the runner it
replaces: linux.arm64.m7g.4xlarge has no device either. The rocm rows build ROCm
wheels on an ordinary CPU builder and need no AMD hardware.

Depends on pytorch/test-infra#8789, which also carries the two fixes
build_wheels_linux.yml needs to run on an OSDC pod at all.

Authored with Claude Code.
@huydhn
huydhn force-pushed the osdc/wheels-on-osdc branch from f35477c to 2948f6c Compare September 12, 2026 22:57
Drop before merging. Points both the matrix generator and the wheel build at
that PR's branch, and the test-infra checkout with them, so CI here exercises
runner-fleet and the two OSDC fixes it carries. Against test-infra main the
runner-fleet input does not exist and the call is rejected outright.
@huydhn
huydhn marked this pull request as ready for review September 12, 2026 23:01
jobs:
generate-matrix:
uses: pytorch/test-infra/.github/workflows/generate_binary_build_matrix.yml@main
uses: pytorch/test-infra/.github/workflows/generate_binary_build_matrix.yml@build-wheels-linux-osdc-workspace

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @huydhn

Can you share more on how these refs should be updated upon releases? I was usually changing main into e.g. release/2.14 both here and below for test-infra-ref.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants