Merge pull request #98721 from adtac/sprefunsched

scheduler_perf: remove implicit barrier at the end
This commit is contained in:
Kubernetes Prow Robot 2021-02-04 02:02:28 -08:00 committed by GitHub
commit 35bc0fbad5
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -254,8 +254,7 @@ func (cpo createPodsOp) patchParams(w *workload) (realOp, error) {
// barrierOp defines an op that can be used to wait until all scheduled pods of
// one or many namespaces have been bound to nodes. This is useful when pods
// were scheduled with SkipWaitToCompletion set to true. A barrierOp is added
// at the end of each each workload automatically.
// were scheduled with SkipWaitToCompletion set to true.
type barrierOp struct {
// Must be "barrier".
Opcode string
@ -411,11 +410,8 @@ func runWorkload(b *testing.B, tc *testCase, w *workload) []DataItem {
b.Fatalf("op %d: invalid op %v", opIndex, concreteOp)
}
}
if err := waitUntilPodsScheduled(ctx, podInformer, b.Name(), nil, numPodsScheduledPerNamespace); err != nil {
// Any pending pods must be scheduled before this test can be considered to
// be complete.
b.Fatal(err)
}
// Some tests have unschedulable pods. Do not add an implicit barrier at the
// end as we do not want to wait for them.
return dataItems
}