-
Notifications
You must be signed in to change notification settings - Fork 83
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
Override node20 compatible glibc for linux x86_64 #5959
base: main
Are you sure you want to change the base?
Conversation
@atalman is attempting to deploy a commit to the Meta Open Source Team on Vercel. A member of the Team first needs to authorize it. |
if: ${{ inputs.architecture == 'x86_64' }} | ||
run: | | ||
yum install curl -yyq | ||
curl -LO https://unofficial-builds.nodejs.org/download/release/v20.9.0/node-v20.9.0-linux-x64-glibc-217.tar.xz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If possible, let's host this on our S3 bucket as I'm not sure how reliably this host is, we might end up ddos it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think actions/runner#2906 (comment) says that rehosting the file is not what nodejs allows people to do, so it's better to just stick with this link then
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! We are using Linux ephemeral runner to build Linux wheel, so I think tampering with the runner is fine as it will be teardown after the job anyway.
Do the same as pytorch/test-infra#5959 and download an old nodejs to keep Ubuntu Bionic working. Bug: actions/checkout#1809 Bug: actions/runner#3373
Do the same as pytorch/test-infra#5959 and download an old nodejs to keep Ubuntu Bionic working. Bug: actions/checkout#1809 Bug: actions/runner#3373 (cherry picked from commit 54cab09)
This is based on this nodejs: https://github.com/nodejs/unofficial-builds/ build compiled with glibc 2.17