fix: correct env var names in Docker execution docs - #870
Merged
Merged
Conversation
Replace non-existent OH_EXECUTION_* env vars with the correct OH_CONVERSATION_* names that match the actual codebase: - OH_EXECUTION_RUNTIME → OH_CONVERSATION_RUNTIME - OH_EXECUTION_IMAGE → OH_CONVERSATION_IMAGE - Remove OH_EXECUTION_PLATFORM (not configurable) - Remove OH_EXECUTION_VOLUMES (not configurable via env var) - Add real container config vars: MEMORY, CPUS, PIDS_LIMIT, STARTUP_TIMEOUT Fixes #869 Co-authored-by: openhands <openhands@all-hands.dev>
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
Verified against software-agent-sdk docker_runtime/registry.py and config.py: - Each container is a full agent-server (OH_CONVERSATION_RUNTIME=local, own secret key, session API key and persistence) proxied by the outer server; drop the DockerExecutionWorkspace / execution-only endpoint / fixed tool list description. - Containers always bind-mount the conversation dir, a persistence dir and the host workspace, so files and history persist; remove the ephemeral and no-mounts claims. - Fill in real defaults (4g, 2.0 CPUs, 512 PIDs, 120s startup) and document OH_CONVERSATION_IDLE_TTL_SECONDS (20 min). - Fix the verification steps (agent-server-conversation- names, three mounts). - Update title/description and the llms.txt entries. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
neubig
approved these changes
Oct 3, 2026
Co-authored-by: openhands <openhands@all-hands.dev>
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary of changes
The Docker execution docs page (
openhands/usage/agent-canvas/backend-setup/docker-execution.mdx) referenced environment variables that do not exist anywhere in the codebase. This PR replaces them with the correctOH_CONVERSATION_*variable names that match the actual implementation.Changes:
OH_EXECUTION_RUNTIME→OH_CONVERSATION_RUNTIMEOH_EXECUTION_IMAGE→OH_CONVERSATION_IMAGEOH_EXECUTION_PLATFORM— platform is not configurable via env varOH_EXECUTION_VOLUMES— volumes are not configurable via env varlinux/arm64platform noteOH_CONVERSATION_CONTAINER_MEMORYOH_CONVERSATION_CONTAINER_CPUSOH_CONVERSATION_CONTAINER_PIDS_LIMITOH_CONVERSATION_CONTAINER_STARTUP_TIMEOUTFixes #869
This PR was created by an AI agent (OpenHands) on behalf of @DevinVinson.