-
Notifications
You must be signed in to change notification settings - Fork 24.9k
Move inductor workflows focal (ubuntu 20.04) -> jammy (ubuntu 22.04) #154153
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/154153
Note: Links to docs will display an error until the docs builds have been completed. ❌ 3 New Failures, 26 Pending, 3 Unrelated FailuresAs of commit f5fa877 with merge base 413664b ( NEW FAILURES - The following jobs have failed:
BROKEN TRUNK - The following jobs failed but were present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
TIL, we don't need to manually create new ECR entries anymore, which is super nice |
d70b171
to
4c0e74e
Compare
@pytorchmergebot merge -f "all required signal is green" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
@pytorchbot revert -m "Broke inductor tests, see https://hud.pytorch.org/hud/pytorch/pytorch/b8452e55bc50109a14add60397fee6c85486ae8d/1?per_page=50&name_filter=inductor_torchbench&mergeEphemeralLF=true" -c nosignal |
@pytorchbot successfully started a revert job. Check the current status here. |
… 22.04) (#154153)" This reverts commit 2bd95f3. Reverted #154153 on behalf of https://github.com/malfet due to Broke inductor tests, see https://hud.pytorch.org/hud/pytorch/pytorch/b8452e55bc50109a14add60397fee6c85486ae8d/1?per_page=50&name_filter=inductor_torchbench&mergeEphemeralLF=true ([comment](#154153 (comment)))
@atalman your PR has been successfully reverted. |
Looks like it broke opacus_cifar10: #154446 |
maybe this is just a new release of opacus? https://pypi.org/project/opacus/#history |
The fact this is an eager failure is pretty suspicious given the nature of your PR,
|
This does look like opacus update: Previous passing PR: https://github.com/pytorch/pytorch/actions/runs/15272660608/job/42954922171#step:22:1478 |
@pytorchmergebot merge -f "lint is green, this is a reland" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
…4437) Fixes #154157 Inductor Workflows where moved from focal to jammy here: #154153 Pull Request resolved: #154437 Approved by: https://github.com/Skylion007, https://github.com/cyyever, https://github.com/davidberard98, https://github.com/huydhn
…orch#154437) Fixes pytorch#154157 Inductor Workflows where moved from focal to jammy here: pytorch#154153 Pull Request resolved: pytorch#154437 Approved by: https://github.com/Skylion007, https://github.com/cyyever, https://github.com/davidberard98, https://github.com/huydhn
Trying to fix: #154157