From e8bb82cdf021adc228fecdf67f50498478720386 Mon Sep 17 00:00:00 2001 From: Mike Spreitzer Date: Mon, 17 Jun 2024 17:37:44 -0400 Subject: [PATCH] Increase noise margin for APF integration test Signed-off-by: Mike Spreitzer --- .../apiserver/flowcontrol/concurrency_util_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/integration/apiserver/flowcontrol/concurrency_util_test.go b/test/integration/apiserver/flowcontrol/concurrency_util_test.go index 7d5fa348b37..6b1efd6984e 100644 --- a/test/integration/apiserver/flowcontrol/concurrency_util_test.go +++ b/test/integration/apiserver/flowcontrol/concurrency_util_test.go @@ -280,9 +280,9 @@ func TestConcurrencyIsolation(t *testing.T) { // standard deviation divided by mean, for a class of traffic is a characterization of all the noise that applied to // that class. We found that noxu1 generally had a much bigger CV than noxu2. This makes sense, because noxu1 probes // more behavior --- the waiting in queues. So we take the minimum of the two as an indicator of the relative amount - // of noise that comes from all the other behavior. Currently, we use 2 times the experienced coefficient of variation + // of noise that comes from all the other behavior. Currently, we use 3 times the experienced coefficient of variation // as the margin of error. - margin := 2 * math.Min(noxu1LatStats.cv, noxu2LatStats.cv) + margin := 3 * math.Min(noxu1LatStats.cv, noxu2LatStats.cv) t.Logf("Error margin is %v", margin) isConcurrencyExpected := func(name string, observed float64, expected float64) bool {