Skip to content

Override node20 compatible glibc for linux x86_64 - #5959

Closed
atalman wants to merge 7 commits into
pytorch:mainfrom
atalman:more_modifications_to_job
Closed

atalman wants to merge 7 commits into
pytorch:mainfrom
atalman:more_modifications_to_job

Conversation

@atalman

@atalman atalman commented Nov 21, 2024 •

Copy link
Copy Markdown
Contributor

@vercel

vercel Bot commented Nov 21, 2024

Copy link
Copy Markdown

@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.

@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 Nov 21, 2024
@atalman atalman changed the title test Override node20 compatible glibc for linux x86_64 Nov 21, 2024
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

Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Contributor

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

@huydhn huydhn left a comment

Copy link
Copy Markdown
Contributor

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.

thesamesam added a commit to thesamesam/meson that referenced this pull request Dec 10, 2024
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
eli-schwartz pushed a commit to mesonbuild/meson that referenced this pull request Dec 15, 2024
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)
hoilc added a commit to hoilc/mycli-standalone that referenced this pull request Jan 2, 2025
hoilc added a commit to hoilc/elasticsearch-dump-standalone that referenced this pull request Jan 2, 2025
@atalman

atalman commented Apr 25, 2025

Copy link
Copy Markdown
Contributor Author

not rquired any more closing

@atalman atalman closed this Apr 25, 2025
Volker-Weissmann pushed a commit to Volker-Weissmann/meson that referenced this pull request May 1, 2025
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
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.

4 participants