-
Notifications
You must be signed in to change notification settings - Fork 13.4k
npm GETs in CI can be flaky, can we cache them? #142433
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
Comments
#142434 should fix this. It should indeed be preinstalled, if possible. |
@Kobzol Unfortunately it seems we have more than that tidy job pinging npm, though I agree it is the priority to fix: #142422 (comment) |
seems we also want to preinstall a browser-ui-test version? + head -n 1 /tmp/browser-ui-test.version
+ npm install [email protected] --unsafe-perm=true |
huh... rust/src/ci/docker/host-x86_64/x86_64-gnu-tools/Dockerfile Lines 79 to 105 in 6c8138d
|
Yeah, but now we just have to wait until half the internet comes back again. |
true, hard to cache anything if you can't get it in the first place. |
current status of outage at https://status.cloud.google.com/
|
idk if it's worth it but I thought I'd raise the subject.
see #142432
cc @Kobzol @GuillaumeGomez
The text was updated successfully, but these errors were encountered: