Skip to content

[Manylinux 2.28] Correct Linux aarch64 cuda binaries wheel name#150786

Closed
atalman wants to merge 2 commits intopytorch:mainfrom
atalman:fix_aarch64_filename
Closed

[Manylinux 2.28] Correct Linux aarch64 cuda binaries wheel name#150786
atalman wants to merge 2 commits intopytorch:mainfrom
atalman:fix_aarch64_filename

Conversation

@atalman
Copy link
Copy Markdown
Contributor

@atalman atalman commented Apr 7, 2025

Related to: #149044 (comment)
For CPU binaries we run auditwheel however for cuda binaries auditwheel produces invalid results . Hence we need to rename the file.

@atalman atalman requested a review from a team as a code owner April 7, 2025 20:34
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot bot commented Apr 7, 2025

🔗 Helpful Links

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

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

❌ 1 New Failure, 28 Pending

As of commit c829d90 with merge base f8b53f4 (image):

NEW FAILURE - The following job has failed:

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

Copy link
Copy Markdown
Contributor

@malfet malfet left a comment

Choose a reason for hiding this comment

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

This is probably fine, but I don't see a reason why we should not rename the wheel for CPU version as well...

@atalman atalman added topic: not user facing topic category ciflow/binaries Trigger all binary build and upload jobs on the PR labels Apr 7, 2025
@atalman
Copy link
Copy Markdown
Contributor Author

atalman commented Apr 8, 2025

@pytorchmergebot merge -f "lint is green and all required tests"

@pytorchmergebot
Copy link
Copy Markdown
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
Copy link
Copy Markdown
Contributor Author

atalman commented Apr 8, 2025

@pytorchbot cherry-pick --onto release/2.7 -c critical

pytorchbot pushed a commit that referenced this pull request Apr 8, 2025
)

Related to: #149044 (comment)
For CPU binaries we run auditwheel however for cuda binaries auditwheel produces invalid results . Hence we need to rename the file.
Pull Request resolved: #150786
Approved by: https://github.com/malfet

(cherry picked from commit 836955b)
@pytorchbot
Copy link
Copy Markdown
Collaborator

Cherry picking #150786

The cherry pick PR is at #150820 and it is recommended to link a critical cherry pick PR with an issue. The following tracker issues are updated:

Details for Dev Infra team Raised by workflow job

atalman added a commit that referenced this pull request Apr 8, 2025
)

[Manylinux 2.28] Correct Linux aarch64 cuda binaries wheel name (#150786)

Related to: #149044 (comment)
For CPU binaries we run auditwheel however for cuda binaries auditwheel produces invalid results . Hence we need to rename the file.
Pull Request resolved: #150786
Approved by: https://github.com/malfet

(cherry picked from commit 836955b)

Co-authored-by: atalman <atalman@fb.com>
timocafe pushed a commit to timocafe/pytorch that referenced this pull request Apr 16, 2025
…rch#150786)

Related to: pytorch#149044 (comment)
For CPU binaries we run auditwheel however for cuda binaries auditwheel produces invalid results . Hence we need to rename the file.
Pull Request resolved: pytorch#150786
Approved by: https://github.com/malfet
amathewc pushed a commit to amathewc/pytorch that referenced this pull request Apr 17, 2025
…rch#150786)

Related to: pytorch#149044 (comment)
For CPU binaries we run auditwheel however for cuda binaries auditwheel produces invalid results . Hence we need to rename the file.
Pull Request resolved: pytorch#150786
Approved by: https://github.com/malfet
atalman added a commit to pytorch/test-infra that referenced this pull request Apr 18, 2025
Same as pytorch/pytorch#150786
Wheel filename should be:
linux_aarch64 -> manylinux_2_28_aarch64
linux_x86_64 -> manylinux_2_28_x86_64

Test in CI
atalman added a commit to atalman/test-infra that referenced this pull request Apr 18, 2025
Same as pytorch/pytorch#150786
Wheel filename should be:
linux_aarch64 -> manylinux_2_28_aarch64
linux_x86_64 -> manylinux_2_28_x86_64

Test in CI
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.

4 participants