zozo123 commented on PR #71725: URL: https://github.com/apache/airflow/pull/71725#issuecomment-5774118270
Rebased onto current `main`, which reorganised the provider docs twice today: `docs/sandbox.rst`, `docs/toolsets.rst` and `docs/choosing_a_toolset.rst` were replaced by `docs/sandbox/` and `docs/toolsets/`, so the backend is now documented in `docs/sandbox/backends.rst` beside Modal and `sbx` rather than in `toolsets.rst`. Also in this push: the temporary `boat-sdk` entry in `[tool.uv.exclude-newer-package]` is removed from both maps, since its own comment said to once the rolling window advanced past 2026-09-16 and it has; `boat-sdk 1.0.0` now resolves without an override. `main` still claimed in `sbx.py` that no hosted backend ships with the provider, which Modal had already made untrue, so that docstring and the system-test README -- which never mentioned the Modal test -- are corrected here. The system test passed end to end against boat.dev (`1 passed in 14.58s`): create, the marker passed through `SandboxSpec.env` and read back from inside the sandbox, successful and non-zero command exits, write/read/list, and teardown. It requests open egress, because Boat can enforce neither a deny-all policy nor a per-domain allowlist and `create` refuses both rather than provisioning something weaker than the spec asked for. 41 unit tests, ruff and the applicable prek hooks pass locally; mypy is clean on the changed files with the SDK installed. --- Drafted-by: Claude Code (Opus 5); reviewed by @zozo123 before posting -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
