Skip to content

Comments

Skip cooldowned providers during model failover#2143

Merged
gumadeiras merged 2 commits intoopenclaw:mainfrom
YiWang24:fix/failover-cooldown-skip
Jan 27, 2026
Merged

Skip cooldowned providers during model failover#2143
gumadeiras merged 2 commits intoopenclaw:mainfrom
YiWang24:fix/failover-cooldown-skip

Conversation

@YiWang24
Copy link
Contributor

@YiWang24 YiWang24 commented Jan 26, 2026

Based on issue #2142. This PR improves the model failover experience by skipping providers where all auth profiles are currently in cooldown. This avoids the cumulative delay when multiple OAuth-based providers (like google-gemini-cli or google-antigravity) fail or expire simultaneously.

Changes:

  • In runWithModelFallback, we now check the auth store to see if any profiles for the candidate provider are available (not in cooldown).
  • If no profiles are available, the provider is skipped immediately, allowing the system to fall through to stable API key-based providers (like zai) without artificial delays.

@YiWang24
feat(agents): skip cooldowned providers during failover
e5117f9 
When all auth profiles for a provider are in cooldown, the failover
mechanism now skips that provider immediately rather than attempting
and waiting for the cooldown error. This prevents long delays when
multiple OAuth providers fail in sequence.
@YiWang24
fix(agents): correct imports and API usage for cooldown check
5a6aa42
@sebslight sebslight added gateway Gateway runtime and removed gateway Gateway runtime labels Jan 26, 2026
@gumadeiras gumadeiras self-requested a review January 27, 2026 02:09
gumadeiras added a commit that referenced this pull request Jan 27, 2026
@gumadeiras
fix: skip cooldowned providers during model failover (#2143) (thanks @…
a087540 
…YiWang24)
@gumadeiras gumadeiras merged commit ff42a48 into openclaw:main Jan 27, 2026
23 of 25 checks passed
@YiWang24 YiWang24 deleted the fix/failover-cooldown-skip branch January 27, 2026 03:04
gumadeiras added a commit that referenced this pull request Jan 27, 2026
@gumadeiras
fix: skip cooldowned providers during model failover (#2143) (thanks @…
afdcb7a 
…YiWang24)
@gumadeiras gumadeiras mentioned this pull request Jan 27, 2026
Merged
gumadeiras added a commit that referenced this pull request Jan 27, 2026
@gumadeiras
Agents: finish cooldowned provider skip (#2534)
959ddae 
* Agents: skip cooldowned providers in fallback

* fix: skip cooldowned providers during model failover (#2143) (thanks @YiWang24)
@gumadeiras
Copy link
Member

gumadeiras commented Jan 27, 2026

Landed via squash on main.

  • Gate: pnpm lint && pnpm build && pnpm test
  • Merge commit: ff42a48

Follow-up: #2534 adds per-agent auth store checks, rate_limit reason, call-site agentDir, tests, and changelog.

Thanks @YiWang24!

@gumadeiras gumadeiras mentioned this pull request Jan 27, 2026
Closed
tyler6204 added a commit that referenced this pull request Jan 27, 2026
@adam91holt @zerone0x
@claude @Alg0rix @steipete @shakkernerd @orlyjamie @mertcicekci0 @rhuanssauro @thewilloftheshadow @YuriNachos @Alex-Alaniz @kentaro @dguido @obviyus @Glucksberg @vignesh07 @alexstyl @hirefrank @czekaj @jaydenfyi @ppamment @Suksham-sharma @dlauer @tyler6204 @AdeboyeDN @robbyczgw-cla @dominicnunez @gumadeiras @techboss @Kiwitwitter @David-Marsh-Photo @chenyuan99 @jogi47 @YiWang24 @0oAstro @kennyklee
Add per-sender group tool policies and fix precedence (#1757)