diff options
| author | Zach Koopmans <zkoopmans@google.com> | 2020-12-11 12:35:42 -0800 |
|---|---|---|
| committer | gVisor bot <gvisor-bot@google.com> | 2020-12-11 12:37:28 -0800 |
| commit | 305a456551467623d38acdb5a1b9741cb0dc1219 (patch) | |
| tree | 71d2fd82dcb7197b91948440c3fd9e04b4759909 /test/benchmarks/network/node_test.go | |
| parent | 4cba3904f414775371f86571a549454aafad19bf (diff) | |
Adjust requests to be constant equal to b.N in network tests.
For "hey", requests >= concurrency. b.N can be set by the
--test.benchtime={b.N}x. The previous setting of b.N * c
can be surprisingly slow for larger c.
Set the requests to max(b.N, c) and log to the user if it is c.
PiperOrigin-RevId: 347053675
Diffstat (limited to 'test/benchmarks/network/node_test.go')
| -rw-r--r-- | test/benchmarks/network/node_test.go | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/test/benchmarks/network/node_test.go b/test/benchmarks/network/node_test.go index 246720d43..3e837a9e4 100644 --- a/test/benchmarks/network/node_test.go +++ b/test/benchmarks/network/node_test.go @@ -42,9 +42,14 @@ func BenchmarkNode(b *testing.B) { if err != nil { b.Fatalf("Failed to parse parameters: %v", err) } + requests := b.N + if requests < c { + b.Logf("b.N is %d must be greater than threads %d. Consider running with --test.benchtime=Nx where N >= %d", b.N, c, c) + requests = c + } b.Run(name, func(b *testing.B) { hey := &tools.Hey{ - Requests: b.N * c, // Requests b.N requests per thread. + Requests: requests, Concurrency: c, } runNode(b, hey) |