From 4105fea6eb0cf1e5dd56146ccf94755445988b9a Mon Sep 17 00:00:00 2001 From: draveness Date: Wed, 4 Sep 2019 09:41:06 +0800 Subject: [PATCH] fix(scheduler): remove the defer function cost --- test/integration/scheduler_perf/scheduler_bench_test.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/test/integration/scheduler_perf/scheduler_bench_test.go b/test/integration/scheduler_perf/scheduler_bench_test.go index 90e32f89ebb..451ba429547 100644 --- a/test/integration/scheduler_perf/scheduler_bench_test.go +++ b/test/integration/scheduler_perf/scheduler_bench_test.go @@ -21,7 +21,7 @@ import ( "testing" "time" - "k8s.io/api/core/v1" + v1 "k8s.io/api/core/v1" storagev1beta1 "k8s.io/api/storage/v1beta1" "k8s.io/apimachinery/pkg/api/resource" metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" @@ -409,6 +409,9 @@ func benchmarkScheduling(numNodes, numExistingPods, minPods int, // Since the total amount of time is relatively large, it might not be a concern. time.Sleep(100 * time.Millisecond) } + + // Note: without this line we're taking the overhead of defer() into account. + b.StopTimer() } // makeBasePodWithSecrets creates a Pod object to be used as a template.