From: Benny Siegert <bsiegert@gmail.com>
Date: Thu, 12 Apr 2018 14:32:40 +0000 (+0200)
Subject: net/http: address comments from previous review
X-Git-Tag: go1.11beta1~857
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=7a0bf9436960ffb0b2bd6c7421cd420aa8577402;p=gostls13.git

net/http: address comments from previous review

Move the test skip to use testenv.SkipFlaky and link to the Go issue.

Update #24826

Change-Id: I7a0ea3325ffcaa790b25f8cdc429fb52e96a41c7
Reviewed-on: https://go-review.googlesource.com/106636
Reviewed-by: Tobias Klauser <tobias.klauser@gmail.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
---

diff --git a/src/net/http/serve_test.go b/src/net/http/serve_test.go
index 05e31ef1e4..49239b3671 100644
--- a/src/net/http/serve_test.go
+++ b/src/net/http/serve_test.go
@@ -5630,8 +5630,7 @@ func runTimeSensitiveTest(t *testing.T, durations []time.Duration, test func(t *
 // read if it's already done one.
 func TestServerDuplicateBackgroundRead(t *testing.T) {
 	if runtime.GOOS == "netbsd" && runtime.GOARCH == "arm" {
-		// http://gnats.netbsd.org/53173
-		t.Skip("skipping to avoid crash")
+		testenv.SkipFlaky(t, 24826)
 	}
 
 	setParallel(t)