Create a swapfile in workflow-run-job-linux

[trxcllnt]

Create a swapfile in workflow-run-job-linux to work around OOMs during compiling.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: de8c8845-bcb4-45ff-bfb0-d937cba46f47

📥 Commits

Reviewing files that changed from the base of the PR and between 0ed347e and 9c1e455.

📒 Files selected for processing (1)

• .github/actions/workflow-run-job-linux/action.yml

🚧 Files skipped from review as they are similar to previous changes (1)

• .github/actions/workflow-run-job-linux/action.yml

---

📝 Walkthrough

Summary by CodeRabbit

• Bug Fixes
  • Improved reliability for Linux-based automated runs by initializing temporary swap space when needed for memory-intensive jobs, helping prevent failures.
  • Adjusted system swappiness settings after enabling swap to better handle low-memory scenarios.

Walkthrough

Added Linux CI swap provisioning to the generated ci.sh run step. The script now measures disk and memory, skips creation when swap is already active, otherwise creates /swapfile, enables it, and sets vm.swappiness=10.

Changes

Swap Provisioning in Linux CI Action

Layer / File(s)	Summary
Swap provisioning logic .github/actions/workflow-run-job-linux/action.yml	Adds disk/memory calculation, swap detection, conditional /swapfile creation via fallocate, swap enabling, and vm.swappiness=10 in the generated script.

Sequence Diagram(s)

Not applicable.

Suggested labels: ci, infrastructure

Suggested reviewers: N/A

---

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 2

🧹 Nitpick comments (2)

.github/actions/workflow-run-job-linux/action.yml (2)

117-117: 🩺 Stability & Availability | 🔵 Trivial | ⚡ Quick win

suggestion: fallocate isn't supported on all filesystems (e.g., some overlay/network filesystems used by CI runners) and fails with "fallocate failed: Operation not supported". Given set -e is active, this would abort the job outright. Consider falling back to dd if fallocate fails.

Proposed fallback

-          sudo fallocate -l "$((SWAP_SIZE / 1024 / 1024 / 1024))G" /swapfile
+          sudo fallocate -l "$((SWAP_SIZE / 1024 / 1024 / 1024))G" /swapfile || \
+            sudo dd if=/dev/zero of=/swapfile bs=1M count=$((SWAP_SIZE / 1024 / 1024))

---

116-116: 🎯 Functional Correctness | 🔵 Trivial | 💤 Low value

suggestion: if DISK_SIZE / 4 rounds down to 0 (very small root disk), fallocate -l 0G /swapfile may fail or create a useless empty swapfile. Worth a minimum-size floor, though likely low risk on real CI hardware.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 925bb416-855e-4993-85d3-e15c1e66a15e

📥 Commits

Reviewing files that changed from the base of the PR and between 84c2c01 and 0ed347e.

📒 Files selected for processing (1)

• .github/actions/workflow-run-job-linux/action.yml

[Jacobfaib]

@trxcllnt rebasing over main should fix the clang-tidy issue.

Are the swap files ephemeral to the docker images? Or permanent? If they are ephemeral, perhaps we should consider allocating them directly in the devcontainer builds if that's possible as a longer-term solution.

[bernhardmgruber]

/ok to test 9e2b7ae

[github-actions]

😬 CI Workflow Results

🟥 Finished in 1h 34m: Pass: 99%/506 | Total: 5d 07h | Max: 1h 25m | Hits: 93%/730740

See results here.