Skip to content

ci: set TMPDIR=/host/tmp in datapath verifier tests#45047

Merged
aanm merged 1 commit intomainfrom
pr/fix-lvh-error-out-of-space
Mar 30, 2026
Merged

ci: set TMPDIR=/host/tmp in datapath verifier tests#45047
aanm merged 1 commit intomainfrom
pr/fix-lvh-error-out-of-space

Conversation

@aanm
Copy link
Copy Markdown
Member

@aanm aanm commented Mar 30, 2026

Use /var/tmp for Go build temporary files to avoid running out of
disk space on the tmpfs-backed /tmp in the LVH VM. Also redirect the
extract-llvm.sh output to /var/tmp for the same reason.
/var/tmp lives on the VM's real disk (not tmpfs) and supports mmap
(unlike the 9p-mounted /host), making it suitable for Go's linker.

@aanm aanm requested a review from julianwiedmann March 30, 2026 07:55
@aanm aanm requested review from a team as code owners March 30, 2026 07:55
@aanm aanm added needs-backport/1.17 This PR / issue needs backporting to the v1.17 branch needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch labels Mar 30, 2026
@aanm aanm requested a review from viktor-kurchenko March 30, 2026 07:55
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Mar 30, 2026
@aanm aanm added kind/bug/CI This is a bug in the testing code. release-note/ci This PR makes changes to the CI. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Mar 30, 2026
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Mar 30, 2026
@aanm aanm enabled auto-merge March 30, 2026 07:55
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Mar 30, 2026

/test

@cilium-ariane
Copy link
Copy Markdown

cilium-ariane bot commented Mar 30, 2026

/test

@aanm aanm force-pushed the pr/fix-lvh-error-out-of-space branch from f17ba5a to c1fff44 Compare March 30, 2026 09:25
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Mar 30, 2026

/ci-verifier

@aanm aanm marked this pull request as draft March 30, 2026 09:26
auto-merge was automatically disabled March 30, 2026 09:26

Pull request was converted to draft

@cilium-ariane
Copy link
Copy Markdown

cilium-ariane bot commented Mar 30, 2026

/test

Use /var/tmp for Go build temporary files to avoid running out of
disk space on the tmpfs-backed /tmp in the LVH VM. Also redirect the
extract-llvm.sh output to /var/tmp for the same reason.
/var/tmp lives on the VM's real disk (not tmpfs) and supports mmap
(unlike the 9p-mounted /host), making it suitable for Go's linker.

Signed-off-by: Andr Martins <andre@cilium.io>
@aanm aanm force-pushed the pr/fix-lvh-error-out-of-space branch from c1fff44 to d733873 Compare March 30, 2026 10:16
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Mar 30, 2026

/ci-verifier

@cilium-ariane
Copy link
Copy Markdown

cilium-ariane bot commented Mar 30, 2026

/test

@aanm aanm marked this pull request as ready for review March 30, 2026 10:52
@aanm aanm enabled auto-merge March 30, 2026 10:52
@aanm aanm added this pull request to the merge queue Mar 30, 2026
Merged via the queue into main with commit 2231bcf Mar 30, 2026
98 of 101 checks passed
@aanm aanm deleted the pr/fix-lvh-error-out-of-space branch March 30, 2026 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug/CI This is a bug in the testing code. needs-backport/1.17 This PR / issue needs backporting to the v1.17 branch needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch release-note/ci This PR makes changes to the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants