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

Bug: CI fails because it can't set up Node #6463

Open
2wheeh opened this issue Jul 26, 2024 · 1 comment
Open

Bug: CI fails because it can't set up Node #6463

2wheeh opened this issue Jul 26, 2024 · 1 comment

Comments

@2wheeh
Copy link
Contributor

2wheeh commented Jul 26, 2024

The current behavior

Unexpectedly, CI fails while fetching NodeJS with a 522 error (timeout):

log: https://github.com/facebook/lexical/actions/runs/10107570076/job/27951708511?pr=6367

image

According to the log, the builds use the cached version from actions/node-versions and skip the download step if the version (i.e., 18.18.0) can be found in manifest.

But 18.18.0 for arm64 is not on this list. So we fetch it directly from Node.js:

image

And there is possibility of hitting rate limit when downloading from dist:

...there is possibility of hitting rate limit when downloading from dist
https://github.com/actions/setup-node#supported-version-syntax

which is leading timeout with 522 unexpectedly.

The expected behavior

No failures while setting up node on CI.

@etrepum
Copy link
Collaborator

etrepum commented Jul 28, 2024

Maybe it makes sense to update the version of node that is used? I'm not sure why it was all on 18 to begin with

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants