[psarna]

What does this PR do?

Adds pty to e2b sandbox provider, so that it is usable with sandbox/shell.py script.

Validation

With local e2b-compatible server:

export E2B_API_KEY=test-key \
       E2B_API_URL=http://127.0.0.1:10129 \
       E2B_SANDBOX_URL=http://127.0.0.1:10129

CFG=$(mktemp -d)/e2b.yaml
cat > "$CFG" <<'EOF'
sandbox:
  e2b:
    connection:
      api_key: ${oc.env:E2B_API_KEY,null}
      api_url: ${oc.env:E2B_API_URL,null}
      sandbox_url: ${oc.env:E2B_SANDBOX_URL,null}
    create:
      template_map:
        "docker.io/library/python:3.12-slim": "docker.io/library/python:3.12-slim"
EOF

.venv/bin/python -m nemo_gym.sandbox.shell --config "$CFG"

Rollouts

Compatibility and benchmark impact

Checklist

• I have read the contributing guidelines.
• The change is focused; unrelated "drive-by" edits are tracked as separate issues/PRs.
• Tests added or updated and pass locally, or N/A for docs-only / non-code changes (so CI unit/server checks pass when applicable).
• Documentation added or updated, or N/A with justification.
• Pre-commit checks pass locally (pre-commit run --all-files) (so CI lint/format checks pass).
• New source files include the required Apache-2.0 SPDX header.
• All commits have DCO sign-off (git commit -s) (so the DCO check passes).

[copy-pr-bot]

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

[psarna]

/ok to test 8757498

[psarna]

/ok to test 7eb8d94