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.

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

🐛 Source Monday: handle complexityBudgetExhausted error #36717

Merged
merged 6 commits into from
Apr 8, 2024

Conversation

darynaishchenko
Copy link
Collaborator

oncall: https://github.com/airbytehq/oncall/issues/4759

error_handler doesn't have a case for complexityBudgetExhausted, which is actually 429 error and source need to retry.

{
    "errors": [
        "Complexity budget exhausted, query cost 222250 budget remaining 214980 out of 10000000 reset in 16 seconds"
    ],
    "status_code": 429,
    "error_message": "Complexity budget exhausted, query cost 222250 budget remaining 214980 out of 10000000 reset in 16 seconds",
    "error_code": "complexityBudgetExhausted",
    "account_id": <obfuscated>
}

Added this case to error_handler in manifest.yaml. Covered this case in integration tests.

@darynaishchenko darynaishchenko self-assigned this Apr 1, 2024
Copy link

vercel bot commented Apr 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 5, 2024 5:03pm

@erohmensing
Copy link
Contributor

Let's close #36741 as well when this is in if it resolves the issue :)

@darynaishchenko darynaishchenko merged commit 560bead into master Apr 8, 2024
31 of 32 checks passed
@darynaishchenko darynaishchenko deleted the daryna/source-monday/key-error-data branch April 8, 2024 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/monday team/connectors-python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants