Conversation
} | ||
catch (Exception ex) | ||
{ | ||
logger.LogError(ex, "Error starting redis docker container, retrying."); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't this be a possible infinite loop?
Do we want a counter and give up after N retries?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It only calls Run
twice, no looping
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, it's not recursive, it just looks kinda like it is because of the local func ;).
Add a delay? Just a second or two.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Synchronous delay :(
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, then don't :(
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually yeah, leave it in. It's OK here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool yeah re-reading I see it now. :)
} | ||
catch (Exception ex) | ||
{ | ||
logger.LogError(ex, "Error starting redis docker container, retrying."); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, it's not recursive, it just looks kinda like it is because of the local func ;).
Add a delay? Just a second or two.
} | ||
catch (Exception ex) | ||
{ | ||
logger.LogError(ex, "Error starting redis docker container, retrying."); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, then don't :(
Fixes #3256
cc @muratg Test only 2.2 change