From 75bcdd59635a33e2a210ef6b02f5e3814571de4b Mon Sep 17 00:00:00 2001 From: "Bryan C. Mills" Date: Tue, 18 Jan 2022 12:21:18 -0500 Subject: [PATCH] net/http: skip TestClientTimeout_Headers_h{1,2} on windows/arm and windows/arm64 This extends the skip added in CL 375635 to the "_Headers" variant of the test, since we have observed similar failures in that variant on the builders. For #43120 Change-Id: Ib1c97fbb776b576271629272f3194da77913a941 Reviewed-on: https://go-review.googlesource.com/c/go/+/379156 Trust: Bryan Mills Run-TryBot: Bryan Mills TryBot-Result: Gopher Robot Reviewed-by: Damien Neil --- src/net/http/client_test.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/net/http/client_test.go b/src/net/http/client_test.go index ea59f68f35..e91d526824 100644 --- a/src/net/http/client_test.go +++ b/src/net/http/client_test.go @@ -1339,6 +1339,9 @@ func testClientTimeout_Headers(t *testing.T, h2 bool) { t.Error("net.Error.Timeout = false; want true") } if got := ne.Error(); !strings.Contains(got, "Client.Timeout exceeded") { + if runtime.GOOS == "windows" && strings.HasPrefix(runtime.GOARCH, "arm") { + testenv.SkipFlaky(t, 43120) + } t.Errorf("error string = %q; missing timeout substring", got) } } -- 2.50.0