Skip to content

[DO NOT MERGE] Smoke test the v3 python fallback - #23361

Closed
huydhn wants to merge 2 commits into
mainfrom
osdc/test-v3-python-fallback
Closed

huydhn wants to merge 2 commits into
mainfrom
osdc/test-v3-python-fallback

Conversation

@huydhn

@huydhn huydhn commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

Testing pytorch/test-infra#8980, which makes linux_job_v3 look for python3 before using it and fall back to uv when the image has none.

ExecuTorch images do have python, so this is the side that must not change — these jobs should take the existing path and behave exactly as on main. tensordict#1807 covers the other side, where the image has no python at all.

Two cheap qnn jobs are pinned (workflow ref and test-infra-ref); the other 42 v3 jobs here stay on main, so any difference is attributable.

Delete this branch once #8980 lands.

Authored with Claude Code.

Do not merge. Pins two cheap qnn jobs at
pytorch/test-infra#8980, both the workflow ref and test-infra-ref.

That branch makes linux_job_v3 look for python3 before using it, and fall
back to uv when the image has none. These images do have python, so this is
the side that must not change: it should take the existing path and behave
exactly as it does on main. tensordict#1807 covers the other side, where the
image has no python at all.

The remaining 42 v3 jobs here stay on main, so any difference is attributable.

Authored with Claude Code.
@pytorch-bot

pytorch-bot Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/23361

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

✅ No Failures

As of commit 2142a16 with merge base 0f008e1 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 2, 2026
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

Authored with Claude Code.
@huydhn

huydhn commented Oct 2, 2026

Copy link
Copy Markdown
Contributor Author

(Claude, on huydhn's behalf.) Closing — pytorch/test-infra#8980 has merged. This was a throwaway smoke test; it confirmed the no-regression side (python present → Install uv skipped, job green).

@huydhn huydhn closed this Oct 2, 2026
@huydhn
huydhn deleted the osdc/test-v3-python-fallback branch October 2, 2026 22:24

This branch was successfully deployed

1 active deployment
cadence — 2142a164 Deployed Oct 2, 2026 by huydhn via hifi-op-test / hifi4 #31138
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant