8000 Use Manylinux 2.28 for nightly build and cxx11-abi by atalman · Pull Request #143423 · pytorch/pytorch · GitHub
[go: up one dir, main page]

Skip to content

Use Manylinux 2.28 for nightly build and cxx11-abi #143423

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

Closed
wants to merge 3 commits into from

Conversation

atalman
Copy link
Contributor
@atalman atalman commented Dec 17, 2024

As per: https://dev-discuss.pytorch.org/t/pytorch-linux-wheels-switching-to-new-wheel-build-platform-manylinux-2-28-on-november-12-2024/2581

Linux Builds: CPU, CUDA 11.8, CUDA 12.4 switched to Manylinux 2.28 and D_GLIBCXX_USE_CXX11_ABI=1 on the week of Dec 16

@atalman atalman requested a review from a team as a code owner December 17, 2024 20:43
@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Dec 17, 2024
Copy link
pytorch-bot bot commented Dec 17, 2024

🔗 Helpful Links

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

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

❌ 7 New Failures, 5 Cancelled Jobs, 2 Unrelated Failures

As of commit 3d99c93 with merge base 7c25a55 (image):

NEW FAILURES - The following jobs have failed:

CANCELLED JOBS - The following jobs were cancelled. Please retry:

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

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

@@ -126,7 +126,7 @@ jobs:
# favor of GPU_ARCH_VERSION
DESIRED_CUDA: cpu
GPU_ARCH_TYPE: cpu
DOCKER_IMAGE: pytorch/manylinux-builder:cpu-main
DOCKER_IMAGE: pytorch/manylinux2_28-builder:cpu-main
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Please note, this is not used on MacOS arm64. TODO: Post a followup to remove this parameter

Copy link
Contributor
@huydhn huydhn left a comment

Choose a reason for hiding this comment

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

Yay, I have been waiting for this

@huydhn
Copy link
Contributor
huydhn commented Dec 17, 2024

cc @q10 I think this would help with the various issues on FBGEMM once we have a new PT nightly after this lands

@atalman atalman added the ciflow/binaries Trigger all binary build and upload jobs on the PR label Dec 17, 2024
@atalman
Copy link
Contributor Author
atalman commented Dec 18, 2024

@pytorchmergebot merge -f "lint and required builds are green"

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

atalman added a commit to pytorch/test-infra that referenced this pull request Dec 18, 2024
Switch nightly ecosystem Wheel Builds to manylinux 2.28

pytorch/pytorch PR: pytorch/pytorch#143423
aditew01 pushed a commit to aditew01/pytorch that referenced this pull request Dec 18, 2024
8000
facebook-github-bot pushed a commit to pytorch/FBGEMM that referenced this pull request Dec 26, 2024
Summary:
X-link: facebookresearch/FBGEMM#612

This reverts commit 5c16f4b.  This is not needed anymore after pytorch/pytorch#143423.  I think this will also fix the issue with building torchrec CPU https://github.com/pytorch/FBGEMM/actions/runs/12470608879/job/34806045264?pr=3528#step:18:219⁩

### Testing

https://github.com/pytorch/FBGEMM/actions/runs/12470608879

Pull Request resolved: #3528

Reviewed By: q10

Differential Revision: D67602736

Pulled By: huydhn

fbshipit-source-id: ec5888acacd96295dd6dfe26e5fa87b28810b2bc
q10 pushed a commit to q10/FBGEMM that referenced this pull request Apr 10, 2025
Summary:
Pull Request resolved: facebookresearch/FBGEMM#612

This reverts commit 5c16f4b.  This is not needed anymore after pytorch/pytorch#143423.  I think this will also fix the issue with building torchrec CPU https://github.com/pytorch/FBGEMM/actions/runs/12470608879/job/34806045264?pr=3528#step:18:219⁩

### Testing

https://github.com/pytorch/FBGEMM/actions/runs/12470608879

X-link: pytorch#3528

Reviewed By: q10

Differential Revision: D67602736

Pulled By: huydhn

fbshipit-source-id: ec5888acacd96295dd6dfe26e5fa87b28810b2bc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/binaries Trigger all binary build and upload jobs on the PR Merged topic: not user facing topic category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0