Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

CLI: Retry on connection errors while acquiring token #2668

Merged
merged 2 commits into from
Dec 2, 2022

Conversation

Porges
Copy link
Member

@Porges Porges commented Dec 2, 2022

While running tests we sometimes see token acquisition fail due to "Connection reset by peer". This should be retried by default.

@Porges Porges force-pushed the retry-acquire-token branch from 3b22a86 to 585e48b Compare December 2, 2022 00:29
@Porges Porges force-pushed the retry-acquire-token branch from 585e48b to 2ac1bb1 Compare December 2, 2022 00:45
@Porges Porges changed the title Retry on connection errors while acquiring token CLI: Retry on connection errors while acquiring token Dec 2, 2022
@codecov-commenter
Copy link

codecov-commenter commented Dec 2, 2022

Codecov Report

Merging #2668 (eb10357) into main (be066bd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2668   +/-   ##
=======================================
  Coverage   29.29%   29.29%           
=======================================
  Files         289      289           
  Lines       35755    35755           
=======================================
  Hits        10474    10474           
  Misses      25281    25281           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Porges Porges merged commit 4aec5ed into main Dec 2, 2022
@Porges Porges deleted the retry-acquire-token branch December 2, 2022 03:30
@nharper285 nharper285 mentioned this pull request Dec 2, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Jan 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants