From 0d944767ecd150f4ae8dc685dbc5668d34486a91 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. Kubernetes-commit: fa68e8248594e54cda4cc06ab69030f6267ebbb7 --- rest/request_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rest/request_test.go b/rest/request_test.go index 30ae87d8f2..8e9bb92b5a 100644 --- a/rest/request_test.go +++ b/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{},