Skip to content

fix: remove --test_tmpdir that conflicts with --sandbox_tmpfs_path #23

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.

Sign up for GitHub

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

Merged
Sam-Si merged 4 commits into from
May 10, 2026
Merged

fix: remove --test_tmpdir that conflicts with --sandbox_tmpfs_path #23

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
94d1d4a
fix: remove --test_tmpdir that conflicts with --sandbox_tmpfs_path
Sam-Si May 9, 2026
bd0e7a0
fix: use /dev/shm as sandbox base to prevent stale 'File exists' errors
Sam-Si May 9, 2026
6090042
fix: resolve sandbox 'File exists' errors without /dev/shm
Sam-Si May 9, 2026
fb0f2f6
fix: remove --sandbox_debug from default test flags
Sam-Si May 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 7 additions & 2 deletions .bazelrc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,10 +44,15 @@ build --jobs=20
build --local_resources=cpu=16
build --local_resources=memory=57344

# Linux Sandbox Hardening: Fixes "File exists" and "/dev/null" errors
# Linux Sandbox Hardening
# --sandbox_tmpfs_path=/tmp : clean /tmp per action (isolation)
# --sandbox_add_mount_pair : ensures /dev/null is available
# NOTE: Do NOT use --sandbox_base=/dev/shm — Docker containers default
# to 64MB /dev/shm which is too small for linking large binaries.
# Stale sandbox state ("File exists" errors) is handled by cleaning
# .bazel/output_base/sandbox before builds (see dcodex-setup.sh).
build:linux --sandbox_tmpfs_path=/tmp
build:linux --sandbox_add_mount_pair=/dev/null
build:linux --test_tmpdir=/tmp/bazel-test-logs
build:linux --dynamic_mode=off
build:linux --linkopt=-Wl,--threads=16

Expand Down
15 changes: 14 additions & 1 deletion dcodex-setup.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -309,6 +309,17 @@ else
ok "Skipping bazel clean (incremental build — disk cache preserved)"
fi

# Always purge stale sandbox directories. If a previous build was interrupted
# (Ctrl+C, OOM kill, crash), leftover files cause "File exists" errors on the
# next run. This is cheap (~instant) and only removes sandbox working dirs —
# the disk cache and repo cache are untouched.
if [[ -d "${REPO_DIR}/.bazel/output_base/sandbox" ]]; then
rm -rf "${REPO_DIR}/.bazel/output_base/sandbox"
ok "Purged stale sandbox directories"
else
ok "No stale sandbox directories to clean"
fi

timer

# ─────────────────────────────────────────────────────────────────────────────
Expand Down Expand Up @@ -350,9 +361,11 @@ timer
step "6/7 Tests"

# Common Bazel test flags for diagnostics — always verbose.
# NOTE: --sandbox_debug is intentionally omitted; it dumps per-action
# traces for every compile/link step, drowning test output. Pass it
# manually if debugging sandbox issues: bazel test --sandbox_debug ...
BAZEL_TEST_COMMON=(
--verbose_failures
--sandbox_debug
--test_output=all
--test_env=HOME=/tmp
)
Expand Down
Loading