Skip to content

Core: tiny fixes for retry logic#2106

Merged
jtopjian merged 2 commits intogophercloud:masterfrom
kayrus:retry
Feb 2, 2021
Merged

Core: tiny fixes for retry logic#2106
jtopjian merged 2 commits intogophercloud:masterfrom
kayrus:retry

Conversation

@kayrus
Copy link
Copy Markdown
Contributor

@kayrus kayrus commented Jan 31, 2021

Tiny fixes for #2096
Support Swift 498 response code

@coveralls
Copy link
Copy Markdown

coveralls commented Jan 31, 2021

Coverage Status

Coverage remained the same at 79.788% when pulling 3aac7b7 on kayrus:retry into 8e76a0a on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Jan 31, 2021

Build succeeded.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Feb 1, 2021

Build succeeded.

Copy link
Copy Markdown
Contributor

@jtopjian jtopjian left a comment

Choose a reason for hiding this comment

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

LGTM - thank you!

@jtopjian
Copy link
Copy Markdown
Contributor

jtopjian commented Feb 1, 2021

@kayrus To confirm, this is ready to go? Just want to make sure. And after merging this, I'll merge gophercloud/utils#144.

@kayrus
Copy link
Copy Markdown
Contributor Author

kayrus commented Feb 1, 2021

@jtopjian yes, it is ready. once you merge this, I need to update dependencies in gophercloud/utils#144

@jtopjian jtopjian merged commit 633d735 into gophercloud:master Feb 2, 2021
@kayrus kayrus deleted the retry branch February 2, 2021 06:51
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.

3 participants