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

ci(node): just rate limit #2440

Closed
greyscaled opened this issue Jan 17, 2024 · 4 comments · Fixed by #2476
Closed

ci(node): just rate limit #2440

greyscaled opened this issue Jan 17, 2024 · 4 comments · Fixed by #2476
Assignees
Labels
build Build or release related ci Related to CI/CD

Comments

@greyscaled
Copy link
Contributor

run: https://github.com/GlareDB/glaredb/actions/runs/7560954189

Error:

Status: Downloaded newer image for ghcr.io/napi-rs/napi-rs/nodejs-rust:lts-debian
+ curl --proto =https --tlsv1.2 -sSf https://just.systems/install.sh
+ bash -s -- --to .
curl: (22) The requested URL returned error: 403

Cause:

curl --proto '=https' --tlsv1.2 -sSf https://just.systems/install.sh | bash -s -- --to . &&
./just protoc &&

Referring to https://just.systems/man/en/chapter_5.html it seems like we need to add --tag:

Note that install.sh may fail on GitHub actions, or in other environments where many machines share IP addresses. install.sh calls GitHub APIs in order to determine the latest version of just to install, and those API calls are rate-limited on a per-IP basis. To make install.sh more reliable in such circumstances, pass a specific tag to install with --tag.

@greyscaled greyscaled added build Build or release related ci Related to CI/CD labels Jan 17, 2024
@scsmithr
Copy link
Member

Fixed in #2442

@greyscaled
Copy link
Contributor Author

Still seeing this

@greyscaled greyscaled reopened this Jan 22, 2024
@tychoish
Copy link
Contributor

@greyscaled link to a recent failure?

@greyscaled
Copy link
Contributor Author

@greyscaled greyscaled linked a pull request Jan 22, 2024 that will close this issue
@tychoish tychoish self-assigned this Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Build or release related ci Related to CI/CD
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants