provider: drop --gpu-mode from CAPE run face (flag removed upstream; pool-level sharing)#159
Merged
Merged
Conversation
…pool-level sharing) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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
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.
Upstream change
CAPE removed the
cape run --gpu-modeflag entirely: requests now inherit exclusive/shared GPU behavior from the target pool, with no per-request override. In the same release,cape statusgained three cotenancy fields:gpu_mode(derived,null|"exclusive"|"shared"),gpu_cotenant_count, andgpu_cotenant_counts.Failure mode without this fix
The provider unconditionally emitted
--gpu-mode wholeon everycape run. Against the current CLI that is an unknown-argument error — argparse exits 2 with a usage message before anything is submitted — so everycreate()fails and no sandbox can be created at all.Changes
agentix/provider/cape.py: drop the--gpu-mode wholepair from the run argv and from the verified-contract docstring.tests/test_cape_provider.py:--gpu-mode, matching the real parser by construction (unknown flag → rc=2, usage on stderr).test_fake_cape_rejects_dropped_gpu_mode_flagdocuments the exit-2 behavior of the removed flag.test_run_face_never_emits_dropped_gpu_mode_flagregression test asserts the run argv never contains--gpu-mode(gpu unset and gpu=4).--gpu-mode wholepresent to--gpu-modeabsent.cape statusnow emits the three new cotenancy fields, proving the provider's status parsing tolerates (ignores) them — it only readsstateand therequest_idecho, so no parsing change was needed.README.md: flag list updated; Contract status section notes the upstream flag removal (pool-level exclusive/shared) and the new status cotenancy fields tolerated by our parser.Gates
uv run ruff check .— cleanuv run pyright— 0 errorsuv run pytest plugins/providers/cape/tests -q— 31 passeduv run pytest tests/ plugins/ -q— greenuv lock --check— up to date🤖 Generated with Claude Code