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

Use a larger runner for the release process #446

Merged
merged 1 commit into from
Dec 19, 2024
Merged

Use a larger runner for the release process #446

merged 1 commit into from
Dec 19, 2024

Conversation

zanieb
Copy link
Member

@zanieb zanieb commented Dec 19, 2024

This has been failing repeatedly due to disk space, the hope is that this resolves it

This has been failing repeatedly due to disk space, the hope is that this resolves it
@zanieb zanieb added internal A refactor or improvement that is not user-facing ci:skip labels Dec 19, 2024
@zanieb zanieb marked this pull request as ready for review December 19, 2024 00:46
@zanieb zanieb merged commit 47cc679 into main Dec 19, 2024
10 checks passed
@zanieb zanieb deleted the zb/release branch December 19, 2024 04:09
@@ -26,7 +26,7 @@ jobs:
release:
env:
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
runs-on: ubuntu-latest
runs-on: ubuntu-latest-large
Copy link
Contributor

Choose a reason for hiding this comment

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

You're likely aware there is a bunch of space taken up by default frameworks on the runners, but to add punch to that, I'll note Pex uses this: https://github.com/pex-tool/pex/blob/ed020784733d1233944774e9f1052490a48a28d0/.github/workflows/ci.yml#L111-L123

And it nets ~14GB currently: https://github.com/pex-tool/pex/actions/runs/12403252596/job/34628545010#step:2:326

I'm not sure if that is enough savings to help your case.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks! That's good to know.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm a fan of the jlumbroso/free-disk-space action.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:skip internal A refactor or improvement that is not user-facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants