github/golang.net
2
0
Fork 0
mirror of https://github.com/golang/net.git synced 2026-03-31 10:27:08 +09:00
Code Issues Packages Projects Releases Wiki Activity

netutil: in TestLimitListenerSaturation, allow some connections to fail to dial

Browse Source 
When the listener saturates, the kernel will typically buffer the
remaining pending connections (so they will eventually succeed in
dialing). However, that behavior is not guaranteed, and it empirically
doesn't always hold: a failure was observed in
https://build.golang.org/log/5ac7312814bcff4841563be043f28aaefa9b3c90.

We do expect to be able to dial at least as many connections as the
listener will accept, and we expect every connection that is accepted
to be served to completion.

Updates golang/go#22926

Change-Id: I4cb39c8f39fda0dcb905f548612ccdf1856f2a66
Reviewed-on: https://go-review.googlesource.com/c/net/+/380155
Trust: Bryan Mills <bcmills@google.com>
Run-TryBot: Bryan Mills <bcmills@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
This commit is contained in:
Bryan C. Mills
2022-01-21 14:27:22 -05:00
committed by Bryan Mills
parent 2ed6ce1e17
commit e204ce36a2
1 changed files with 11 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
netutil/listen_test.go
View File

@@ -203,11 +203,17 @@ func TestLimitListenerSaturation(t *testing.T) {
wg.Wait()
t.Logf("served %d connections (of %d dialed, %d attempted)", served, dialed, attemptsPerWave)
// We expect that the kernel can queue at least attemptsPerWave
// connections at a time (since it's only a small number), so every
// connection should eventually be served.
if served != attemptsPerWave {
t.Errorf("expected %d served", attemptsPerWave)
// Depending on the kernel's queueing behavior, we could get unlucky
// and drop one or more connections. However, we should certainly
// be able to serve at least max attempts out of each wave.
// (In the typical case, the kernel will queue all of the connections
// and they will all be served successfully.)
if dialed < max {
t.Errorf("expected at least %d dialed", max)
}
if served < dialed {
t.Errorf("expected all dialed connections to be served")
}
}
}