Thanks to visit codestin.com
Credit goes to github.com

Skip to content

CICD.yml: Avoid no space left #9277

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.

Sign up for GitHub

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

Jump to bottom
Merged
sylvestre merged 3 commits into from
Nov 15, 2025
Merged

CICD.yml: Avoid no space left #9277

sylvestre merged 3 commits into from
Nov 15, 2025

Conversation

@oech3
Copy link
Contributor

@oech3 oech3 commented Nov 14, 2025

Try to fix #9276

@oech3 oech3 force-pushed the patch-2 branch 4 times, most recently from 9464bed to e799381 Compare November 14, 2025 17:21
@github-actions
Copy link

github-actions bot commented Nov 14, 2025

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

@github-actions
Copy link

github-actions bot commented Nov 15, 2025

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

@github-actions
Copy link

github-actions bot commented Nov 15, 2025

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

@oech3
Copy link
Contributor Author

oech3 commented Nov 15, 2025

@naoNao89 I think this is not using /tmp. Am I missing something?

@oech3 oech3 force-pushed the patch-2 branch 2 times, most recently from f43242d to 6f94fb4 Compare November 15, 2025 03:33
naoNao89
naoNao89 reviewed Nov 15, 2025
View reviewed changes
@naoNao89
Copy link
Contributor

naoNao89 commented Nov 15, 2025

Verification: Add after cleanup:

- name: Verify disk space
  run: df -h

@github-actions
Copy link

github-actions bot commented Nov 15, 2025

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

@oech3 oech3 force-pushed the patch-2 branch 4 times, most recently from 151d40e to 639939e Compare November 15, 2025 04:48
@oech3
Copy link
Contributor Author

oech3 commented Nov 15, 2025

Maybe, is line to do rm -rf wrong?

@github-actions
Copy link

github-actions bot commented Nov 15, 2025

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

@oech3 oech3 force-pushed the patch-2 branch 4 times, most recently from 6a09c78 to 725c79d Compare November 15, 2025 05:38
@codspeed-hq
Copy link

codspeed-hq bot commented Nov 15, 2025 •
edited
Loading

CodSpeed Performance Report

Merging #9277 will improve performances by 2.28%

Comparing oech3:patch-2 (1dd9848) with main (5202ac1)

Summary

⚡ 1 improvement
✅ 122 untouched
⏩ 5 skipped1

Benchmarks breakdown

Benchmark BASE HEAD Change
⚡ tsort_input_parsing_heavy[5000] 84.4 ms 82.5 ms +2.28%

Footnotes

  1. 5 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

@oech3 oech3 force-pushed the patch-2 branch 2 times, most recently from 73182f4 to b33288a Compare November 15, 2025 06:08
@oech3
Copy link
Contributor Author

oech3 commented Nov 15, 2025

Looks fine! Also needed to fix #9278

@github-actions
Copy link

github-actions bot commented Nov 15, 2025

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

@github-actions
Copy link

github-actions bot commented Nov 15, 2025

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

@oech3 oech3 mentioned this pull request Nov 15, 2025
Closed
sylvestre
sylvestre reviewed Nov 15, 2025
View reviewed changes
.github/workflows/CICD.yml Outdated
- name: build and test all features individually
shell: bash
run: |
command -v sudo && sudo rm -rf /usr/share/dotnet #/usr/local/lib/android /usr/share/swift /usr/lib/firefox /opt/google/chrome /opt/microsoft/msedge # avoid no space left
Copy link
Contributor

@sylvestre sylvestre Nov 15, 2025

Choose a reason for hiding this comment

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

please remove the commented path

Copy link
Contributor Author

@oech3 oech3 Nov 15, 2025

Choose a reason for hiding this comment

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

Removed.

@oech3 oech3 requested a review from sylvestre November 15, 2025 09:17
@github-actions
Copy link

github-actions bot commented Nov 15, 2025

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)

@sylvestre sylvestre merged commit 8fc9d28 into uutils:main Nov 15, 2025
125 of 126 checks passed
@oech3 oech3 deleted the patch-2 branch November 15, 2025 12:58
naoNao89 pushed a commit to naoNao89/coreutils that referenced this pull request Nov 24, 2025
@oech3 @naoNao89
Merge pull request uutils#9277 from oech3/patch-2
a5ce771 
CICD.yml: Avoid no space left
romanstingler pushed a commit to romanstingler/coreutils that referenced this pull request Dec 14, 2025
@oech3 @romanstingler
Merge pull request uutils#9277 from oech3/patch-2
adf0dde 
CICD.yml: Avoid no space left
@moonfruit moonfruit mentioned this pull request Dec 15, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sylvestre sylvestre Awaiting requested review from sylvestre

1 more reviewer

@naoNao89 naoNao89 naoNao89 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

CICD / Test all features separately: many No space left on device

3 participants

@oech3 @naoNao89 @sylvestre