-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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-36947: [CI] Move free up disk space to the Jinja macros to be able to reuse it on docs job #36948
Conversation
@github-actions crossbow submit test-ubuntu-22.04-docs |
@github-actions crossbow submit -g linux |
Revision: aa823b2 Submitted crossbow builds: ursacomputing/crossbow @ actions-a80b3bfb36
|
Revision: aa823b2 Submitted crossbow builds: ursacomputing/crossbow @ actions-35c61b8c04 |
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
I think that this is also useful for docs and other jobs.
@github-actions crossbow submit test-ubuntu-22.04-docs |
@github-actions crossbow submit -g linux |
Revision: 16a2e44 Submitted crossbow builds: ursacomputing/crossbow @ actions-7c9afc39a2
|
Revision: 16a2e44 Submitted crossbow builds: ursacomputing/crossbow @ actions-8b267107a1 |
… to reuse it on docs job (#36948) ### Rationale for this change Try to get rid of some failures on docs generation on release and reuse existing code. ### What changes are included in this PR? Move step to a macro to be able to reuse it ### Are these changes tested? Archery tasks ### Are there any user-facing changes? No * Closes: #36947 Authored-by: Raúl Cumplido <[email protected]> Signed-off-by: Raúl Cumplido <[email protected]>
After merging your PR, Conbench analyzed the 5 benchmarking runs that have been run so far on merge-commit 66a6136. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. It also includes information about possible false positives for unstable benchmarks that are known to sometimes produce them. |
…e able to reuse it on docs job (apache#36948) ### Rationale for this change Try to get rid of some failures on docs generation on release and reuse existing code. ### What changes are included in this PR? Move step to a macro to be able to reuse it ### Are these changes tested? Archery tasks ### Are there any user-facing changes? No * Closes: apache#36947 Authored-by: Raúl Cumplido <[email protected]> Signed-off-by: Raúl Cumplido <[email protected]>
…e able to reuse it on docs job (apache#36948) ### Rationale for this change Try to get rid of some failures on docs generation on release and reuse existing code. ### What changes are included in this PR? Move step to a macro to be able to reuse it ### Are these changes tested? Archery tasks ### Are there any user-facing changes? No * Closes: apache#36947 Authored-by: Raúl Cumplido <[email protected]> Signed-off-by: Raúl Cumplido <[email protected]>
Rationale for this change
Try to get rid of some failures on docs generation on release and reuse existing code.
What changes are included in this PR?
Move step to a macro to be able to reuse it
Are these changes tested?
Archery tasks
Are there any user-facing changes?
No