test(route): cover 429 tool continuation after free failover - #1282
seonghobae wants to merge 3 commits into
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
… into test/1016-tool-429-continuation
|
Exact-head admission correction — Ready is review admission only. Fresh audit against base
This PR is moved to Draft/Proposed until the causal owner repair is present on a successor exact head and re-audited. Queued/pending work is neither an additional blocker nor passing evidence. No Close, force push, destructive rebase, manual rerun, synthetic status/approval, merge, auto-merge, or bypass was performed. |
Scope
Extend #1279's real HTTP, two-request
orchestrator/freetool-session check to a provider-declared 429 with finite retry timing. The first request must advance to an eligible free worker, return its tool call and typed attempt receipt, and accept the caller's tool result on the next request through the serving worker.This is one parameterized acceptance case over the existing 502 fixture. It changes no production routing policy or retry budget.
Verification
962c4144af480e6013b486e9c5630b3571409fdf.tests/test_actions_model_fallback.py— 34 passed, process exit 0.git diff --checkpassed.Merge boundary
Draft and stacked on #1279. Keep #1279's queued Security run untouched. After the parent reaches protected
main, retarget and require this PR's own exact-head hosted checks and independent review before merging.Refs #1016.
Current review admission — 2026-09-28
This head remains
962c4144af480e6013b486e9c5630b3571409fdfon exact parent #127942d78c55a0e58df8f033a8b58616c2f77897ca8b. Its recorded Python 3.12 focused suite passed 34 tests on this unchanged head. The PR is now Ready so its own non-skipped hosted checks and independent review can run. Parent #1279 is still open with queued checks; local evidence does not approve either PR. The earlier Draft statement is historical.