github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-09-01 09:18:45 +00:00
Code Issues Projects Releases Wiki Activity

Use random backoff for retries in cloud-cidr-allocator

Browse Source
This commit is contained in:
Shyam Jeedigunta
2018-08-31 14:56:53 +02:00
parent ff46c0fe6b
commit 8123a9ac7b
2 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
pkg/controller/nodeipam/ipam/cloud_cidr_allocator.go
View File

@@ -18,6 +18,7 @@ package ipam
import (
"fmt"
"math/rand"
"net"
"sync"
"time"
@@ -211,9 +212,9 @@ func nodeUpdateRetryTimeout(count int) time.Duration {
timeout *= 2
}
if timeout > maxUpdateRetryTimeout {
return maxUpdateRetryTimeout
timeout = maxUpdateRetryTimeout
}
return timeout
return time.Duration(timeout.Nanoseconds()/2 + rand.Int63n(timeout.Nanoseconds()))
}
func (ca *cloudCIDRAllocator) removeNodeFromProcessing(nodeName string) {

6
pkg/controller/nodeipam/ipam/cloud_cidr_allocator_test.go
View File

@@ -59,6 +59,10 @@ func TestBoundedRetries(t *testing.T) {
}
}
func withinExpectedRange(got time.Duration, expected time.Duration) bool {
return got >= expected/2 && got <= 3*expected/2
}
func TestNodeUpdateRetryTimeout(t *testing.T) {
for _, tc := range []struct {
count int
@@ -71,7 +75,7 @@ func TestNodeUpdateRetryTimeout(t *testing.T) {
{count: 50, want: 5000 * time.Millisecond},
} {
t.Run(fmt.Sprintf("count %d", tc.count), func(t *testing.T) {
if got := nodeUpdateRetryTimeout(tc.count); got != tc.want {
if got := nodeUpdateRetryTimeout(tc.count); !withinExpectedRange(got, tc.want) {
t.Errorf("nodeUpdateRetryTimeout(tc.count) = %v; want %v", got, tc.want)
}
})