Skip to content
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

Jump to bottom

[Bug]: Inconsistent default SANDBOX_RUNTIME_CONTAINER_IMAGE value #4749

Open
1 task done
zetomatoz opened this issue Nov 5, 2024 · 1 comment · May be fixed by #4750
Open
1 task done

[Bug]: Inconsistent default SANDBOX_RUNTIME_CONTAINER_IMAGE value #4749

zetomatoz opened this issue Nov 5, 2024 · 1 comment · May be fixed by #4750
Labels
bug Something isn't working

Comments

@zetomatoz
Copy link

Is there an existing issue for the same bug?

  • I have checked the existing issues.

Describe the bug and reproduction steps

Current Behavior

The SANDBOX_RUNTIME_CONTAINER_IMAGE environment variable has inconsistent values across the documentation and compose.yml files (i.e runtime:0.9-nikolaik instead of runtime:0.12-nikolaik). This discrepancy can cause confusing runtime error messages that are difficult to troubleshoot. Additionally, the documentation does not clearly emphasize that the sandbox runtime’s source code version must be synchronized with the backend to prevent errors.

Expected Behavior

When OpenHands releases a new version, the SANDBOX_RUNTIME_CONTAINER_IMAGE value should be updated and consistent across all parts of the repository. The documentation should also explicitly state the importance of keeping the sandbox runtime and backend versions in sync.

OpenHands Installation

Docker command in README

OpenHands Version

0.10, 0.11, 0.12

Operating System

None

Logs, Errors, Screenshots, and Additional Context

No response
@zetomatoz zetomatoz added the bug Something isn't working label Nov 5, 2024
Likhithsai2580 added a commit to Likhithsai2580/OpenHands that referenced this issue Nov 5, 2024
@Likhithsai2580
Fix sandbox runtime container image inconsistency
6eb691a 
Fixes All-Hands-AI#4749

Update `SANDBOX_RUNTIME_CONTAINER_IMAGE` value to be consistent across the repository.

* **compose.yml**
  - Update `SANDBOX_RUNTIME_CONTAINER_IMAGE` to `runtime:0.12-nikolaik`.

* **containers/dev/compose.yml**
  - Update `SANDBOX_RUNTIME_CONTAINER_IMAGE` to `runtime:0.12-nikolaik`.

* **docs/modules/usage/installation.mdx**
  - Add a note emphasizing the importance of synchronizing the sandbox runtime and backend versions.

* **README.md**
  - Add a note emphasizing the importance of synchronizing the sandbox runtime and backend versions.
@Likhithsai2580 Likhithsai2580 linked a pull request Nov 5, 2024 that will close this issue
Open
@mamoodi
Copy link
Collaborator

mamoodi commented Nov 5, 2024

Thanks. The docker compose was written by a community member but is not actively supported. We will try to update it going forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix sandbox runtime container image inconsistency Likhithsai2580/OpenHands
2 participants
@zetomatoz @mamoodi