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 502 errors one time #921

Merged
merged 1 commit into from
Mar 22, 2021
Merged

Retry 502 errors one time #921

merged 1 commit into from
Mar 22, 2021

Conversation

outcatcher
Copy link
Contributor

@outcatcher outcatcher commented Mar 19, 2021 •
edited
Loading

Summary of the Pull Request

Update gophertelekomcloud to version add retryable 502

Add retry test

PR Checklist

Unittest Steps Performed

=== RUN   TestRoundTripperRetry
--- PASS: TestRoundTripperRetry (0.50s)
PASS

Process finished with exit code 0

@outcatcher
Fix frequently happening 502 error
07898c0 
Update `gophertelekomcloud` to version fix retryable 502

Add retry test
@outcatcher outcatcher requested review from a team, lego963 and anton-sidelnikov and removed request for a team March 19, 2021 20:44
@outcatcher outcatcher changed the title Fix frequently happening 502 error Retry 502 errors one time Mar 22, 2021
@outcatcher outcatcher added the gate Merge PR label Mar 22, 2021
@otc-zuul
Copy link

otc-zuul bot commented Mar 22, 2021

Build failed.

@otc-zuul
Copy link

otc-zuul bot commented Mar 22, 2021

Build succeeded.

@otc-zuul otc-zuul bot merged commit 8a226af into devel Mar 22, 2021
@otc-zuul otc-zuul bot deleted the gw-retry branch March 22, 2021 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lego963 lego963 lego963 approved these changes

@otc-zuul otc-zuul otc-zuul approved these changes

@anton-sidelnikov anton-sidelnikov anton-sidelnikov approved these changes

Assignees
No one assigned
Labels
dependencies gate Merge PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@outcatcher @lego963 @anton-sidelnikov