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

Build on Github Action test-macOS frequently fail due machine run out of disk space #3878

Closed
jakecastelli opened this issue Aug 30, 2024 · 5 comments

Comments

@jakecastelli
Copy link

The Github Actions start to fail frequently on test-macOS build step due to the current build on mac is about 18GB which seems more than the allocated disk space on the runner (14GB limit IIRC). See this for a failure example:

test-macOS
You are running out of disk space. The runner will stop working when the machine runs out of disk space. Free space left: 47 MB

This currently impacts all the Pull Requests on nodejs cores.

@richardlau
Copy link
Member

GitHub only offers 14 GB of storage on their "standard" runners: https://docs.github.com/en/actions/using-github-hosted-runners/using-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories

Unlike for Linux and Windows, GitHub do not offer more storage on their "large" (i.e. paid) runners (still 14 GB for macOS): https://docs.github.com/en/actions/using-github-hosted-runners/using-larger-runners/about-larger-runners#specifications-for-general-larger-runners

Apparently the amount of available disk space available decreased (albeit still with 14 GB available and therefore meeting GH's documented available storage) with GH's 20240827.4 macOS runner image -- there's a general discussion around this over in: actions/runner-images#10511

I'm aware of various hacks/scripts for the Ubuntu runners to reclaim space (see actions/runner-images#10386) but not for macOS.

I do not know if there is anything we can do to reduce the size of Node.js macOS builds -- they appear to be significantly larger than their Linux equivalents.

@targos
Copy link
Member

targos commented Aug 30, 2024

Should we disable macOS jobs for now?

@targos
Copy link
Member

targos commented Aug 30, 2024

The problem is that we would have 0 CI coverage for arm64 macOS if we do it (Jenkins CI currently only tests on x64)

@jakecastelli
Copy link
Author

Do we need Android sdk for macOS arm64 build? Any chance we can temporarily remove it (with a quick patch) from the test-macOS job? seems like it can claim back ~12GB storage space.

jakecastelli added a commit to jakecastelli/node that referenced this issue Aug 30, 2024
nodejs-github-bot pushed a commit to nodejs/node that referenced this issue Aug 30, 2024
Refs: nodejs/build#3878
PR-URL: #54658
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
RafaelGSS pushed a commit to nodejs/node that referenced this issue Aug 30, 2024
Refs: nodejs/build#3878
PR-URL: #54658
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
RafaelGSS pushed a commit to nodejs/node that referenced this issue Aug 30, 2024
Refs: nodejs/build#3878
PR-URL: #54658
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
RafaelGSS pushed a commit to nodejs/node that referenced this issue Aug 30, 2024
Refs: nodejs/build#3878
PR-URL: #54658
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
RafaelGSS pushed a commit to nodejs/node that referenced this issue Aug 31, 2024
Refs: nodejs/build#3878
PR-URL: #54658
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
RafaelGSS pushed a commit to nodejs/node that referenced this issue Aug 31, 2024
Refs: nodejs/build#3878
PR-URL: #54658
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
sendoru pushed a commit to sendoru/node that referenced this issue Sep 1, 2024
Refs: nodejs/build#3878
PR-URL: nodejs#54658
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
RafaelGSS pushed a commit to nodejs/node that referenced this issue Sep 1, 2024
Refs: nodejs/build#3878
PR-URL: #54658
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
@targos
Copy link
Member

targos commented Sep 1, 2024

Fixed by nodejs/node#54658

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

No branches or pull requests

3 participants