Skip to content

ci: skip disk cleanup when unnecessary - #38778

Merged
silverwind merged 2 commits into
go-gitea:mainfrom
silverwind:ci-disk-cleanup-gate
Aug 5, 2026
Merged

ci: skip disk cleanup when unnecessary#38778
silverwind merged 2 commits into
go-gitea:mainfrom
silverwind:ci-disk-cleanup-gate

Conversation

@silverwind

@silverwind silverwind commented Aug 4, 2026

Copy link
Copy Markdown
Member

Speed up all go jobs by ~58s by not cleaning the disk when there's at least 50GB available.

The job was added because some runners only have like 15GB available (I assume self-hosted), but on regular GHA runners which have about 90GB free, this job will now always skip.

The cleanup was added as headroom for the minority of runners that arrive
nearly full, but it ran unconditionally and takes ~58s in every Go job.

Assisted-by: Claude Code:claude-opus-5
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Aug 4, 2026
@silverwind silverwind changed the title ci: skip disk cleanup when the runner has ample space ci: skip disk cleanup when unnecessary Aug 4, 2026
@github-actions github-actions Bot added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Aug 4, 2026
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Aug 5, 2026
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Aug 5, 2026
@silverwind
silverwind merged commit 94199a0 into go-gitea:main Aug 5, 2026
24 checks passed
@GiteaBot GiteaBot added this to the 28.0.0 milestone Aug 5, 2026
@silverwind
silverwind deleted the ci-disk-cleanup-gate branch August 5, 2026 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants