Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Count 'Too Many Requests' as a retriable error #4902

Merged
merged 2 commits into from
Nov 15, 2024
Merged

Count 'Too Many Requests' as a retriable error #4902

merged 2 commits into from
Nov 15, 2024

Conversation

milindl
Copy link
Contributor

@milindl milindl commented Nov 14, 2024

(for OIDC)

@milindl milindl requested a review from a team as a code owner November 14, 2024 19:35
@confluent-cla-assistant
Copy link

🎉 All Contributor License Agreements have been signed. Ready to merge.
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

@airlock-confluentinc airlock-confluentinc bot force-pushed the dev_count_too_many_requests_as_retriable branch from 72187a4 to a47556c Compare November 15, 2024 03:13
@airlock-confluentinc airlock-confluentinc bot force-pushed the dev_count_too_many_requests_as_retriable branch from 7f42fdb to d4809f4 Compare November 15, 2024 12:49
emasab
emasab approved these changes Nov 15, 2024
View reviewed changes
Copy link
Contributor

@emasab emasab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@emasab emasab merged commit cb8c19c into master Nov 15, 2024
1 check passed
@emasab emasab deleted the dev_count_too_many_requests_as_retriable branch November 15, 2024 13:02
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@emasab emasab emasab approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@milindl @emasab