-
Notifications
You must be signed in to change notification settings - Fork 166
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
No space left on test-digitalocean-ubuntu1804-x64-1 #2695
Comments
We have around 3G free:
Space usage:
5.4 G in
I've removed all of
|
Given the nightlies in I think it's perfectly valid to cache headers for recent nightlies, but we probably need a cleanup script of some kind to clean up older ones. |
To clarify, it's not only the node-addon-api jobs that build addons/cause node-gyp to cache headers, but they are the ones that run most frequently (i.e. daily against the nightlies). CITGM will also do so but is run much less often (and hardly at all against nightlies). Metrics: |
Seems to have reoccurred #2877.
Also of note it appears that the headers package is now much larger:
A cursory glance at the changes nodejs/node@8068f40...0cc3720 would suggest that the large increase in size was the OpenSSL 3 switchover. |
@danbev I think you might be the right person to help deal with this:
Also https://nodejs.org/download/release/latest-v16.x/
vs https://nodejs.org/download/release/latest-v17.x/
on the latest 18.x nightly it's even up to 10028309 bytes now! These files were supposed to be small and lean for node-gyp to download and use to compile addons. They're now the size of 0.12 binary tarballs! |
I'm currently on a leave of absence and will be back working in April, and would be happy to take a closer look at this then (unless someone else beats me to it). |
Hit the problem on test-digitalocean-fedora34-x64-2 (reported by @BethGriggs) during https://ci.nodejs.org/job/citgm-smoker/nodes=fedora-latest-x64/2935/.
To reiterate #2695 (comment), we'll need to come up with a script/cleanup job to periodically clear out the cached headers. |
This issue is stale because it has been open many days with no activity. It will be closed soon unless the stale label is removed or a comment is made. |
https://ci.nodejs.org/job/citgm-smoker-nobuild/1103/nodes=ubuntu1804-64/
https://ci.nodejs.org/job/citgm-smoker-nobuild/1102/nodes=ubuntu1804-64/
https://ci.nodejs.org/job/citgm-smoker-nobuild/1101/nodes=ubuntu1804-64/
The text was updated successfully, but these errors were encountered: