Skip to content

fix(sandbox): use an ephemeral root for Blaxel DAX - #272

Open
SystemSculpt wants to merge 1 commit into
computesdk:masterfrom
SystemSculpt:fix/blaxel-dax-ephemeral-root
Open

fix(sandbox): use an ephemeral root for Blaxel DAX#272
SystemSculpt wants to merge 1 commit into
computesdk:masterfrom
SystemSculpt:fix/blaxel-dax-ephemeral-root

Conversation

@SystemSculpt

Copy link
Copy Markdown

Summary

  • Give Blaxel DAX an 8 GiB ephemeral root mounted at /.
  • Require dedicated BL_DAX_API_KEY and BL_DAX_WORKSPACE credentials without a TTI fallback.
  • Keep the existing pooled Blaxel TTI configuration unchanged.
  • Make DAX disk accounting portable across GNU and BusyBox du.
  • Honor the workflow's single-provider dispatch input.

Why

DAX materializes several GiB on the writable root. A memory-backed root reduces the memory available to the workload and can cause an OOM during type checking.

The dedicated root keeps workspace data off RAM. Dedicated credentials also isolate DAX from the pooled TTI workspace.

Validation

  • pnpm install --frozen-lockfile
  • pnpm typecheck
  • pnpm -r --if-present test — 175 passed, 1 skipped
  • actionlint .github/workflows/sandbox-dax-benchmarks.yml
  • bash -n benchmarks/scripts/dax-benchmark.sh
  • ShellCheck for the changed shell behavior
  • Live Blaxel DAX run from this branch:
    • all 7 phases completed;
    • 8 logical CPUs and 16 GiB memory;
    • 8 GiB ephemeral root mounted at /;
    • disk measurements populated after clone, install, and typecheck;
    • zero sandboxes and persistent volumes after cleanup.

Before merge

The benchmark vault must contain non-empty BL_DAX_API_KEY and BL_DAX_WORKSPACE entries. The Blaxel job fails closed when either entry is missing.

@SystemSculpt
SystemSculpt marked this pull request as ready for review July 31, 2026 23:30
@open-cla

open-cla Bot commented Jul 31, 2026

Copy link
Copy Markdown

Contributor License Agreement

All contributors are covered by a CLA.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant