Skip to content

Revert "Add 'break' at RedifshClient request re-try loop"#4861

Merged
yadvr merged 1 commit into
4.15from
revert-4846-redifsh-retry-break
Mar 24, 2021
Merged

Revert "Add 'break' at RedifshClient request re-try loop"#4861
yadvr merged 1 commit into
4.15from
revert-4846-redifsh-retry-break

Conversation

@yadvr

@yadvr yadvr commented Mar 24, 2021

Copy link
Copy Markdown
Member

Reverts #4846

@yadvr

yadvr commented Mar 24, 2021

Copy link
Copy Markdown
Member Author

Unit test failures seen:

Testing timeout of blocking on modified files - 2021-03-24 02:26:28,626 WARN  [utils.nio.Link] (NioTestServer-SSLHandshakeHandler-3:) SSL Handshake has taken more than 30s to connect to: /127.0.0.1:60534. Please investigate this connection.
2021-03-24 02:26:28,635 WARN  [utils.nio.Link] (NioTestServer-SSLHandshakeHandler-4:) SSL Handshake has taken more than 30s to connect to: /127.0.0.1:60536. Please investigate this connection.
pass
[ERROR] Errors: 
[ERROR]   RedfishClientTest.retryHttpRequestExceptionAfterOneRetry »  Unexpected excepti...
[ERROR]   RedfishClientTest.retryHttpRequestExceptionAfterTwoRetries »  Unexpected excep...
[ERROR]   RedfishClientTest.retryHttpRequestNoException:205 » IllegalFormatConversion d ...
[ERROR]   RedfishClientTest.retryHttpRequestSuccessAtTheSecondRetry:227 » IllegalFormatConversion

@GabrielBrascher pl raise a new PR with the unit tests passing.

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.

1 participant