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

Free disk space on GHA worker nodes #19327

Merged
merged 3 commits into from
Oct 10, 2023
Merged

Conversation

wendigo
Copy link
Contributor

@wendigo wendigo commented Oct 10, 2023

No description provided.

@wendigo wendigo requested a review from nineinchnick October 10, 2023 06:22
@cla-bot cla-bot bot added the cla-signed label Oct 10, 2023
@wendigo
Copy link
Contributor Author

wendigo commented Oct 10, 2023

After this operation, 6656 MB disk space will be freed.

Disk space usage:
Filesystem     1K-blocks     Used Available Use% Mounted on
/dev/root       87204404 64193944  22994076  74% /

@wendigo wendigo requested a review from kokosing October 10, 2023 06:39
Copy link
Member

@nineinchnick nineinchnick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shellcheck would complain about missing quotes, but it's good enough and we can improve later.

@wendigo wendigo force-pushed the serafin/clear-up-disk-space branch from 7b9e31a to f1141ed Compare October 10, 2023 07:58
@wendigo wendigo force-pushed the serafin/clear-up-disk-space branch from f1141ed to 0ec2d6c Compare October 10, 2023 08:17
@wendigo wendigo force-pushed the serafin/clear-up-disk-space branch 2 times, most recently from 8008830 to fb58781 Compare October 10, 2023 08:54
@wendigo
Copy link
Contributor Author

wendigo commented Oct 10, 2023

Node after cleanup has now over 50 GB of free space :)

@wendigo wendigo force-pushed the serafin/clear-up-disk-space branch from fb58781 to 9d14e75 Compare October 10, 2023 09:15
@wendigo wendigo force-pushed the serafin/clear-up-disk-space branch 2 times, most recently from 698504f to b8636b7 Compare October 10, 2023 10:09
@wendigo
Copy link
Contributor Author

wendigo commented Oct 10, 2023

Disk space usage before cleaning:
Filesystem     1K-blocks     Used Available Use% Mounted on
/dev/root       87204404 68809452  18378568  79% /
Disk space usage after cleaning:
Filesystem     1K-blocks     Used Available Use% Mounted on
/dev/root       87204404 28003280  59184740  33% /

40 GB reclaimed :)

@wendigo
Copy link
Contributor Author

wendigo commented Oct 10, 2023

This is now enabled per job basis using cleanup-node input for .github/actions/setup

@wendigo wendigo force-pushed the serafin/clear-up-disk-space branch from b8636b7 to 024f8ba Compare October 10, 2023 11:09
This job is fetching various docker images and can easily go out of available disk space
@wendigo wendigo force-pushed the serafin/clear-up-disk-space branch from 024f8ba to 0fe97f8 Compare October 10, 2023 13:50
@wendigo wendigo merged commit 0efef5a into master Oct 10, 2023
94 of 96 checks passed
@wendigo wendigo deleted the serafin/clear-up-disk-space branch October 10, 2023 20:18
@github-actions github-actions bot added this to the 429 milestone Oct 10, 2023
@@ -438,6 +438,7 @@ jobs:
- uses: ./.github/actions/setup
with:
cache: restore
cleanup-node: true
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this the equivalent of docker system prune?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it does more things, see .github/bin/free-disk-space.sh for the impl.

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

Successfully merging this pull request may close these issues.

7 participants