Skip to content

fix(ci): Free Disk Space Reliably#1922

Merged
refcell merged 1 commit intomainfrom
fix/ci-disk-space
Apr 4, 2026
Merged

fix(ci): Free Disk Space Reliably#1922
refcell merged 1 commit intomainfrom
fix/ci-disk-space

Conversation

@refcell
Copy link
Copy Markdown
Contributor

@refcell refcell commented Apr 4, 2026

Summary

Two separate failure modes were causing no-space-left crashes on CI runners. First, the jlumbroso/free-disk-space action uses apt-get internally and silently fails when the disk is already critically full, leaving nothing freed before cache restoration or compilation. This is fixed by adding a fast rm -rf pre-clean step that removes Android SDK, .NET, GHC, Boost, and CodeQL (~15-20 GB) before apt-based cleanup runs. Second, the test-musl-sigsegv job was restoring the shared stable Rust cache, a multi-GB artifact set it has no use for given it only builds one package for a specific target. Removing that cache key eliminates the root cause of its failures.

@refcell refcell added bug Flag: Something isn't working A-ci labels Apr 4, 2026
@refcell refcell self-assigned this Apr 4, 2026
@refcell refcell requested review from danyalprout and jackchuma April 4, 2026 01:11
@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented Apr 4, 2026

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
base Ignored Ignored Apr 4, 2026 1:11am

Request Review

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 4, 2026

No issues found. This is a well-scoped CI fix. The two changes are straightforward and correct:

  1. The fast rm -rf pre-clean runs before the jlumbroso/free-disk-space action, avoiding the circular failure where that action's internal apt-get calls fail when the disk is already critically full. Setting android: false and dotnet: false on the action correctly avoids redundant cleanup of already-removed directories.

  2. Dropping the rust-cache-shared-key from test-musl-sigsegv eliminates unnecessary restoration of a multi-GB cache artifact that this single-package musl build never benefits from.

@refcell refcell added this pull request to the merge queue Apr 4, 2026
Merged via the queue into main with commit ea58ef0 Apr 4, 2026
31 checks passed
@refcell refcell deleted the fix/ci-disk-space branch April 4, 2026 01:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Flag: Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants