Skip to content

ci(codex-review): add retries, concurrency, fallback model#2

Closed
alexanderkreidich wants to merge 1 commit into
mainfrom
fix/codex-review-retries
Closed

ci(codex-review): add retries, concurrency, fallback model#2
alexanderkreidich wants to merge 1 commit into
mainfrom
fix/codex-review-retries

Conversation

@alexanderkreidich

Copy link
Copy Markdown

Summary

  • Drop synchronize trigger; review only on opened/reopened/ready_for_review and skip drafts
  • Add concurrency group per PR with cancel-in-progress: true so old runs are killed
  • urlopen(timeout=120) prevents hangs (root cause of the 2m+ jobs before the 429 landed)
  • Retry 3x on 429 with random(30,60) * attempt sleep; retry 5xx with 5 * attempt
  • Fallback to openai/gpt-4o if primary is still rate-limited after retries
  • Cap diff at 8000 chars (was 12000) with truncation note in the comment

Context

Addressing intermittent HTTP 429 from LiteLLM gateway on openai/gpt-5.3-codex. Previously the job would hang ~2.5min before the gateway rejected with 429; now it fails or falls back predictably.

Test plan

  • Trigger via workflow_dispatch and verify retries log on 429
  • Confirm superseded runs get cancelled when a new PR event fires
  • Verify fallback path runs when primary exhausts retries

@github-actions

Copy link
Copy Markdown

@claude review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant