From 85797eba7075d83b116b5c91ff3b17c2d5118d01 Mon Sep 17 00:00:00 2001 From: Antonio Ojea Date: Mon, 29 Nov 2021 15:37:31 +0100 Subject: [PATCH] bump TestHTTP1DoNotReuseRequestAfterTimeout timeout the test TestHTTP1DoNotReuseRequestAfterTimeout has to wait for request to time out to assert that subsequent requests does not reuse the TCP connection. It seems that current value of 100ms causes issues on some CI environments and bumping the timeout seems to solve this flakiness, We can bump the timeout value because is really low compared to real scenarios and the bump still keeps it in the millisecond order. --- staging/src/k8s.io/client-go/rest/request_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/staging/src/k8s.io/client-go/rest/request_test.go b/staging/src/k8s.io/client-go/rest/request_test.go index 30ae87d8f2e..8e9bb92b5a7 100644 --- a/staging/src/k8s.io/client-go/rest/request_test.go +++ b/staging/src/k8s.io/client-go/rest/request_test.go @@ -2995,7 +2995,7 @@ func TestHTTP1DoNotReuseRequestAfterTimeout(t *testing.T) { config := &Config{ Host: ts.URL, Transport: utilnet.SetTransportDefaults(transport), - Timeout: 100 * time.Millisecond, + Timeout: 1 * time.Second, // These fields are required to create a REST client. ContentConfig: ContentConfig{ GroupVersion: &schema.GroupVersion{},