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

Cherry pick 21de to 1.6 #3730

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

msftrubengu
Copy link
Contributor

@msftrubengu msftrubengu commented Oct 4, 2023

#3718

Microsoft Reviewers: Open in CodeFlow

Change
This pull request honors 429 Retry-After for pre indexed sources and throws ServiceUnavailableException when the response is 429 or 503.

If there's a Retry-After header and its value is more than 60s or the date is more than 60s into the future
Save the next time it can be updated in the source details and not try again for that command.
Subsequent updates or open that source will be skipped if the time lapsed hasn't passed.
If there's a Retry-After header and its value is less than 60s or the date value is less than 60 seconds into the future
Retry once after that amount of time.
If no Retry-After header
Retry after a random amount of time.
For open, the failed to update source message will show up.
For update, a warning with Unavailable will show up.
The next available time will show up in the logs.
@msftrubengu msftrubengu requested a review from a team as a code owner October 4, 2023 22:53
@msftrubengu msftrubengu merged commit e9b25d4 into microsoft:release-v1.6 Oct 4, 2023
5 checks passed
@msftrubengu msftrubengu deleted the cherrypick-21de branch November 14, 2023 00:31
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.

2 participants