Skip to content

Fix OpenEnv docker-image example bootstrap#6329

Merged
qgallouedec merged 3 commits into
mainfrom
fix-openenv-docker-image-bootstrap
Jul 10, 2026
Merged

Fix OpenEnv docker-image example bootstrap#6329
qgallouedec merged 3 commits into
mainfrom
fix-openenv-docker-image-bootstrap

Conversation

@sergiopaniego

@sergiopaniego sergiopaniego commented Jul 8, 2026

Copy link
Copy Markdown
Member

What does this PR do?

Fixes the --env-mode docker-image bootstrap in the OpenEnv catch.py and sudoku.py examples, broken by two OpenEnv changes:

The docker-image bootstrap only needs the container URL, so it now uses LocalDockerProvider().start_container() (the provider exposes a public base_url), mirroring OpenEnv's own examples/browsergym_harness_eval_common.py. Verified end-to-end against a local textarena-env container.

Also drops the docker-hub mode: it relied on Env.from_hub(), which no longer exists in OpenEnv.

Follow-up to #6194 (the .sync() PR, superseded by OpenEnv 0.4.0) and #6138.

Before submitting

AI writing disclosure

  • AI-assisted: some parts were suggested or improved by AI, but the PR was written and reviewed by a human.

Who can review?

@burtenshaw @qgallouedec


Note

Low Risk
Example-only CLI and bootstrap changes; no training library or auth paths affected.

Overview
Repairs --env-mode docker-image in the OpenEnv catch.py and sudoku.py GRPO examples so they work with current OpenEnv instead of broken from_docker_image() / base_url usage.

docker-image now starts the env with LocalDockerProvider: start_container(args.env_image) for the URL, then wait_for_ready(env_url), matching OpenEnv’s recommended container bootstrap.

docker-hub is removed from CLI choices and setup branches because Env.from_hub() is no longer available.

Reviewed by Cursor Bugbot for commit 2994f8f. Bugbot is set up for automated code reviews on this repo. Configure here.

@bot-ci-comment

bot-ci-comment Bot commented Jul 8, 2026

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@qgallouedec
qgallouedec merged commit f572106 into main Jul 10, 2026
13 checks passed
@qgallouedec
qgallouedec deleted the fix-openenv-docker-image-bootstrap branch July 10, 2026 16:11
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.

2 participants