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

Return 429 instead of 503 when all the shards of an index are rate limited #5651

Merged
merged 1 commit into from
Jan 27, 2025
Merged

Return 429 instead of 503 when all the shards of an index are rate limited #5651

merged 1 commit into from
Jan 27, 2025

Conversation

guilload
Copy link
Member

Description

Per title.

How was this PR tested?

Added unit test

@guilload guilload requested a review from rdettai January 26, 2025 21:29
rdettai
rdettai approved these changes Jan 27, 2025
View reviewed changes
Copy link
Collaborator

@rdettai rdettai left a comment

Choose a reason for hiding this comment

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

You might want to revert this: #5566

@rdettai rdettai mentioned this pull request Jan 27, 2025
Closed
@guilload guilload merged commit 99d2c47 into main Jan 27, 2025
8 checks passed
@guilload guilload deleted the guilload/fix-no-shards-available branch January 27, 2025 22:17
This was referenced Jan 28, 2025
Open
Draft
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@rdettai rdettai rdettai 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
@guilload @rdettai