Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Retry on rate limit error #50

Open
mrharpo opened this issue Oct 16, 2024 · 0 comments · May be fixed by #51
Open

Retry on rate limit error #50

mrharpo opened this issue Oct 16, 2024 · 0 comments · May be fixed by #51
Assignees
@mrharpo
Labels
enhancement ➕ New feature or request
Milestone
v0.4.0 Rate limit ♾️

Comments

@mrharpo
Copy link
Contributor

mrharpo commented Oct 16, 2024 •
edited
Loading

HTTP 429

Once we hit SonyCi's rate limits, the API response with HTTPError 429 Too many requests

This is breaking Chowda's sync workflow, since all threads eventually hit this message and are unable to recover.

Retry-After

We should catch this error and implement a retry with a delay according to the received Retry-After header (in seconds)
@mrharpo mrharpo self-assigned this Oct 16, 2024
@mrharpo mrharpo added this to the v0.3.1 Updates ⬆️ milestone Oct 16, 2024
@mrharpo mrharpo added the enhancement ➕ New feature or request label Oct 16, 2024
@mrharpo mrharpo linked a pull request Oct 16, 2024 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mrharpo mrharpo

Labels
enhancement ➕ New feature or request
Projects
Chowda 2024
Status: 🔖 Ready
Milestone
v0.4.0 Rate limit ♾️
Development

Successfully merging a pull request may close this issue.

🔄 Rate limit retry WGBH-MLA/sonyci
1 participant
@mrharpo