Repository navigation
GH-54928: [Packaging][CI] Change TMPDIR for reproducible builds to avoid out of space - #54943
Merged
Merged
Conversation
… to avoid out of space
|
|
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The narrow environment change addresses the reported storage limit without changing build commands or other workflow steps.
No open issues 🎉
What changed in this PR
Addresses GH-54928 by moving Linux package reproducibility checks’ temporary files away from size-limited /tmp storage.
Changes:
- Sets
TMPDIRto${{ runner.temp }}for the APT reproducibility step. - Documents why reprotest needs an alternative temporary directory.
| File | Description |
|---|---|
.github/workflows/package_linux.yml |
Redirects reproducibility-check temporary files to runner-managed storage. |
Review effort: Balanced
raulcd
commented
Oct 23, 2026
Member
Author
|
@kou this should be a quick review :) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TBD
Rationale for this change
Several Linux Packaging jobs are currently failing with out of space on the verify reproducibility step. This is due to
/tmpsize. Ubuntu runner image changed from Ubuntu 24.04 to Ubuntu 26.04 where the/tmpfolder is mounted.In the past the
/tmpfolder was on the root disk which had 145 GB of disk space.With the new runner the
/tmpfolder is ontmpfswith 7.8GB.What changes are included in this PR?
Use
runner.temp(RUNNER_TEMP) to get a temporary directory on the runner instead of using/tmp.See https://docs.github.com/en/actions/reference/workflows-and-actions/contexts and https://docs.github.com/en/actions/reference/workflows-and-actions/variables
Are these changes tested?
Yes via CI
Are there any user-facing changes?
No
Was AI used for this PR?
In accordance to the AI generation guidelines, please disclose below whether and how AI was used in this PR.
PR code and description written by:
Reviewed before submission by: