Skip to content

Commit 49c22f9

Browse files
committed
fix: set tmpfs to 16G, add dockerd startup timeout with error log dump
- Change tmpfs from size=0 (default, could be too small on Modal) to 16G - Replace infinite until loop with 60-second timeout that dumps /var/log/dockerd.log on failure so we can diagnose startup issues - Sandbox now exits cleanly instead of spinning forever if dockerd fails
1 parent 11cd5c2 commit 49c22f9

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

app.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@ def _get_gpu_config(gpu_key: str):
176176
# gVisor requires tmpfs for Docker overlay2 storage driver.
177177
# Without this, Docker falls back to the vfs driver (10-50x slower).
178178
mkdir -p /var/lib/docker
179-
mount -t tmpfs -o size=0 tmpfs /var/lib/docker
179+
mount -t tmpfs -o size=16G tmpfs /var/lib/docker
180180
181181
exec /usr/bin/dockerd --iptables=false --ip6tables=false > /var/log/dockerd.log 2>&1
182182
"""
@@ -740,7 +740,8 @@ async def github_webhook(request: Request):
740740
cmd = (
741741
"bash -c '/start-dockerd.sh &'"
742742
" && sleep 5"
743-
" && until docker info > /dev/null 2>&1; do sleep 1; done"
743+
" && for i in $(seq 1 60); do docker info > /dev/null 2>&1 && break; sleep 1; done"
744+
" && docker info > /dev/null 2>&1 || { echo \"dockerd failed to start\"; cat /var/log/dockerd.log 2>/dev/null; exit 1; }"
744745
" && (docker load -i /images/node-22-alpine.tar || true)"
745746
" && cd /actions-runner"
746747
" && export RUNNER_ALLOW_RUNASROOT=1"

0 commit comments

Comments
 (0)