-
Notifications
You must be signed in to change notification settings - Fork 4k
GH-48807: [CI] Clean up space on GitHub runner to fix manylinux wheel failure #48790
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
GH-48807: [CI] Clean up space on GitHub runner to fix manylinux wheel failure #48790
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
@github-actions crossbow submit wheel-manylinux-2-28-cp312-cp312-amd64 |
|
Revision: 01c6ead Submitted crossbow builds: ursacomputing/crossbow @ actions-df95d87dfc
|
|
The job is passing but there is a really strange timeout: We should probably investigate it as I've been able to reproduce locally with: |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
172652d to
8f9dbbc
Compare
|
@github-actions crossbow submit wheel-manylinux-2-28-cp312-cp312-amd64 |
|
Revision: 8f9dbbc Submitted crossbow builds: ursacomputing/crossbow @ actions-66e2c7e02e
|
This reverts commit 8f9dbbc.
|
@github-actions crossbow submit wheel-manylinux-2-28-cp312-cp312-amd64 |
|
Revision: c1ab74f Submitted crossbow builds: ursacomputing/crossbow @ actions-e409926085
|
|
|
kou
left a comment
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.
+1
|
After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit 2009cc9. There weren't enough matching historic benchmark results to make a call on whether there were regressions. The full Conbench report has more details. |
Rationale for this change
The
wheel-manylinux-2-28-cp312-cp312-amd64performs thealmalinux-8wheels verification and it fails with out of space:At least 746MB more space needed on the / filesystem.What changes are included in this PR?
Use our existing
github_free_spacemacro to clean space on the runner.Are these changes tested?
Yes, via archery. I've validated the failure happened on the PR without the change and was solved with the clean up.
Are there any user-facing changes?
No