Skip to content
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

fix: Download ARM64 binary #5577

Merged
merged 2 commits into from
Aug 21, 2024
Merged

fix: Download ARM64 binary #5577

merged 2 commits into from
Aug 21, 2024

Conversation

zxiiro
Copy link
Collaborator

@zxiiro zxiiro commented Aug 20, 2024

The lambda runner-binaries-syncer is missing code to actually perform the
download for the ARM64 binary despite having the variables in place.
This change resolves that issue so that the syncer actually fetches
the appropriate binary for ARM64.

Closes: pytorch/ci-infra#263
Depends-On: #5569

The runner-binaries-syncer fails with missing module "requests". Seems
like in the past it worked fine without it but is now showing up as
an error.

Relates to pytorch/ci-infra#263.

Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
The lambda runner-binaries-syncer is missing code to actually perform the
download for the ARM64 binary despite having the variables in place.
This change resolves that issue so that the syncer actually fetches
the appropriate binary for ARM64.

Closes: pytorch/ci-infra#263
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Copy link

vercel bot commented Aug 20, 2024

@zxiiro is attempting to deploy a commit to the Meta Open Source Team on Vercel.

A member of the Team first needs to authorize it.

@facebook-github-bot facebook-github-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 Aug 20, 2024
@zxiiro
Copy link
Collaborator Author

zxiiro commented Aug 20, 2024

@jeanschmidt @ZainRizvi

Copy link

vercel bot commented Aug 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
torchci ⬜️ Ignored (Inspect) Visit Preview Aug 20, 2024 10:08pm

@zxiiro
Copy link
Collaborator Author

zxiiro commented Aug 21, 2024

The TF Lint issue doesn't appear to be anything to do with my pull request as far as I can tell.

@zxiiro
Copy link
Collaborator Author

zxiiro commented Aug 21, 2024

Okay I tracked it down and fixed with via #5580.

@ZainRizvi ZainRizvi merged commit 0f0abc8 into pytorch:main Aug 21, 2024
5 of 6 checks passed
@zxiiro zxiiro deleted the arm64-binary branch August 21, 2024 20:45
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.

3 participants