From 2f7f6deef7190d37f70384bc91a40092eea8d5d4 Mon Sep 17 00:00:00 2001 From: Shyam Jeedigunta Date: Thu, 23 Aug 2018 13:26:47 +0200 Subject: [PATCH] Re-calibrate load test throughputs --- test/e2e/scalability/load.go | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/test/e2e/scalability/load.go b/test/e2e/scalability/load.go index 9408a4d8844..33f8ba59e8a 100644 --- a/test/e2e/scalability/load.go +++ b/test/e2e/scalability/load.go @@ -121,10 +121,10 @@ var _ = SIGDescribe("Load capacity", func() { } }) - // We assume a default throughput of 10 pods/second throughput. + // We assume a default throughput of 20 pods/second throughput. // We may want to revisit it in the future. // However, this can be overridden by LOAD_TEST_THROUGHPUT env var. - throughput := 10 + throughput := 20 if throughputEnv := os.Getenv("LOAD_TEST_THROUGHPUT"); throughputEnv != "" { if newThroughput, err := strconv.Atoi(throughputEnv); err == nil { throughput = newThroughput @@ -338,9 +338,10 @@ var _ = SIGDescribe("Load capacity", func() { By("============================================================================") // Cleanup all created replication controllers. - // Currently we assume pods/second average deletion throughput. + // Currently we assume pods/second average deletion throughput, + // as GC is rate-limited and needs to make 2 calls per pod deletion. // We may want to revisit it in the future. - deletingTime := time.Duration(totalPods/throughput) * time.Second + deletingTime := time.Duration(totalPods/(throughput/2)) * time.Second framework.Logf("Starting to delete %v objects...", itArg.kind) deleteAllResources(configs, deletingTime, testPhaseDurations.StartPhase(500, "load pods deletion")) })