Skip to content

Commit

Permalink
do not limit HTTP retries
Browse files Browse the repository at this point in the history
as the tool runs typically on boot, we should not limit the number of
times we poll for conditions to meet our expectations.
  • Loading branch information
Mate Ory committed Apr 30, 2019
1 parent 53d0077 commit 667cdfa
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions cmd/pke/app/util/transport/retry.go
Original file line number Diff line number Diff line change
Expand Up @@ -29,25 +29,21 @@ func NewRetryTransport(rt http.RoundTripper) *rehttp.Transport {
rt,
rehttp.RetryAny(
rehttp.RetryAll(
rehttp.RetryMaxRetries(5),
rehttp.RetryHTTPMethods(http.MethodGet),
rehttp.RetryStatusInterval(400, 600),
),
rehttp.RetryAll(
rehttp.RetryMaxRetries(5),
rehttp.RetryHTTPMethods(http.MethodPost),
rehttp.RetryStatusInterval(500, 600),
),
rehttp.RetryAll(
rehttp.RetryMaxRetries(10),
rehttp.RetryTemporaryErr(),
),
rehttp.RetryAll(
rehttp.RetryMaxRetries(10),
RetryConnectionRefusedErr(),
),
),
rehttp.ExpJitterDelay(2*time.Second, 30*time.Second),
rehttp.ExpJitterDelay(2*time.Second, 60*time.Second),
)
}

Expand Down

0 comments on commit 667cdfa

Please sign in to comment.