Skip to content

Harden GenPlan container runtime - #179

Draft
joshnroy wants to merge 1 commit into
mainfrom
codex/genplan-container-timeout
Draft

joshnroy wants to merge 1 commit into
mainfrom
codex/genplan-container-timeout

Conversation

@joshnroy

@joshnroy joshnroy commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Hardens the GenPlan sandbox staging path for 3D Kindergarden environments.

Changes:

  • Skip bulky Kindergarden mesh/texture asset directories when copying the source tree into per-run staging directories.
  • Bind-mount those asset directories read-only at their expected in-container paths for Docker and Apptainer.
  • Keep blackbox behavior isolated by omitting these source-adjacent assets in blackbox sandboxes.

Why:

  • Copying these assets into every per-replicate sandbox can exhaust /tmp or quota during multi-seed 3D GenPlan runs.
  • Read-only mounts preserve the in-container paths expected by Kindergarden without duplicating large static assets.

Verification:

  • uv run black --check src/robocode/utils/docker_sandbox.py src/robocode/utils/apptainer_sandbox.py tests/utils/test_docker_sandbox.py tests/utils/test_apptainer_sandbox.py
  • uv run isort --check-only src/robocode/utils/docker_sandbox.py src/robocode/utils/apptainer_sandbox.py tests/utils/test_docker_sandbox.py tests/utils/test_apptainer_sandbox.py
  • uv run pytest tests/utils/test_docker_sandbox.py::test_kindergarden_copy_excludes_large_assets tests/utils/test_docker_sandbox.py::test_docker_run_prefix_mounts_kindergarden_assets_read_only tests/utils/test_apptainer_sandbox.py::test_build_cmd_mounts_kindergarden_assets_read_only -q

@joshnroy joshnroy changed the title codex/genplan container timeout Allow ten hours for GenPlan containers Sep 2, 2026
@joshnroy
joshnroy force-pushed the codex/genplan-code-submissions branch from 5323bd1 to ff85c29 Compare September 2, 2026 14:12
@joshnroy

joshnroy commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator Author

Folded into #177 so Claude session reuse and the container wall-clock limit can be reviewed together as one runtime reliability change.

@joshnroy joshnroy closed this Sep 2, 2026
@joshnroy joshnroy reopened this Sep 8, 2026
@joshnroy joshnroy changed the title Allow ten hours for GenPlan containers Harden GenPlan container runtime Sep 8, 2026
@joshnroy
joshnroy force-pushed the codex/genplan-container-timeout branch from 98e26d2 to e0489f6 Compare September 8, 2026 14:21
@joshnroy
joshnroy force-pushed the codex/genplan-code-submissions branch from 414547e to 744d134 Compare September 8, 2026 14:21
@joshnroy
joshnroy force-pushed the codex/genplan-container-timeout branch from e0489f6 to ed64ffd Compare September 8, 2026 14:23
An error occurred while trying to automatically change base from codex/genplan-code-submissions to codex/claude-cli-resume September 8, 2026 16:17
@joshnroy
joshnroy force-pushed the codex/genplan-container-timeout branch from ed64ffd to 1437e9b Compare September 8, 2026 16:52
@joshnroy
joshnroy changed the base branch from codex/genplan-code-submissions to main September 8, 2026 16:53
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