-
-
Notifications
You must be signed in to change notification settings - Fork 229
fix: System.IO.IOException: No space left on device on linux-musl x64 #4743
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
Changes from all commits
3532d9b
f3838b2
e0831ca
d6f5edf
a6fce27
1037435
2dc7ca6
b4486c3
499e7e1
7970193
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -217,6 +217,21 @@ jobs: | |
| build.binlog | ||
| if-no-files-found: ignore | ||
|
|
||
| - name: Free disk space before tests | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Bug: Cleanup action runs inside container, not on runnerThe condition |
||
| if: ${{ contains(matrix.container.image, 'alpine') }} | ||
| run: | | ||
| echo "Disk usage before cleanup:" | ||
| df -h | ||
|
|
||
| # Remove build binlog (can be large, often 100MB+) | ||
| rm -f build.binlog || true | ||
|
|
||
| # Clean up NuGet HTTP cache (not the packages cache which is needed for pack) | ||
| dotnet nuget locals http-cache --clear || true | ||
|
|
||
| echo "Disk usage after cleanup:" | ||
| df -h | ||
|
|
||
| - name: Test | ||
| run: dotnet test ${{ matrix.slnf }} -c Release --no-build --nologo -l GitHubActions -l "trx;LogFilePrefix=testresults_${{ runner.os }}" --collect "XPlat Code Coverage" | ||
|
|
||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
question: We get these warnings during the build:
Can we
if-Condition to only run on supported platforms / not run on non-supported platforms (linux-musl-x64andlinux-musl-arm64)?Maybe by splitting it into a step that does
sudo rm, and another step that does thesudo apt-get(but not for Alpine Linux)?Maybe something like:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's pretty much what we had before:

I'll put that bit back since the freediskspace action is ineffective on the containers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, I meant to just execute the
sudo apt-getconditionally, since they produce the::warning::messages.But keep the
sudo rmon allLinuxrunners, since they seem to work just fine.Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah OK, we can tidy up the non-container runners in another PR if need be 👍🏻