Skip to content

Require workspace storage for microsandbox - #815

Merged
SaladDay merged 1 commit into
mainfrom
require-workspace-storage-for-microsandbox
Oct 21, 2026
Merged

SaladDay merged 1 commit into
mainfrom
require-workspace-storage-for-microsandbox

Conversation

@SaladDay

@SaladDay SaladDay commented Oct 21, 2026 •

Copy link
Copy Markdown
Collaborator

microsandbox now runs every Environment on external workspace storage (the nfs adapter). Its Provider-owned disk mode is gone from the Provider, its helper, Core, Web, the generated projections, the node installer, docs and tests. Docker and E2B keep their own disks.

  • Protocol: DeploymentPolicy.Workspace present means the Provider runs every Environment on external storage. A deployment of such a Provider with no storage selected returns ErrWorkspaceRequired, reported as 409 workspace_storage_required. environment_disk_mib there is only a capacity quota request (default 0).
  • Web: microsandbox setup cannot be saved without workspace storage, and tells the administrator to select it first.
  • mcode does not declare retained_native_history, so the existing rule rejects it on microsandbox. The rejection is now sessions.ErrRetainedHistoryUnsupported, reported as 400 unsupported_or_invalid_configuration with the reason instead of the generic invalid-request message; the compatibility record and Session admission table say so.
  • The node installer creates oac-node with Core's UID 65532 and stops with nothing changed when another account holds that UID or an existing oac-node uses another UID. The manual precreate step is gone from the docs.
  • The microsandbox wire protocol version is 7.

Checks: Core go vet ./...; go test for sandbox, deployment, deploymentpg, execution, api, cmd/server and the whole integration package (its Worker fixture now builds workspace storage as cmd/server does); helper tests with the bundled SDK (scripts/build-microsandbox-provider.py test); pytest deploy/node; agents-client typecheck and tests; Web typecheck and sandbox/i18n tests; translation check. Real-cluster run on mx (together with #816): 409 before storage is selected, the Web hint, oac-node adopted with UID 65532, Codex and Claude Code hosted acceptance 9/9 each, preparation cache and resume probes correct. Playwright cases run in CI only (no browser on the dev server).


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@blacksmith-sh

This comment has been minimized.

@SaladDay
SaladDay force-pushed the require-workspace-storage-for-microsandbox branch from 936397f to 9fae750 Compare October 21, 2026 20:57
@SaladDay
SaladDay merged commit afe91a5 into main Oct 21, 2026
22 checks passed
@SaladDay
SaladDay deleted the require-workspace-storage-for-microsandbox branch October 21, 2026 21:04
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