Merge pull request #67769 from shyamjvs/remove-second-scaling-round-in-load-test

Automatic merge from submit-queue (batch tested with PRs 67748, 67769). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Remove second round of scaling in load test

As it seems redundant to do this twice, not really adding much value. This should cut down ~1.5h in our 5k-node load tests, and helps a bit with https://github.com/kubernetes/test-infra/issues/8348.

/cc @wojtek-t 

```release-note
NONE
```
This commit is contained in:
Kubernetes Submit Queue 2018-08-23 09:20:04 -07:00 committed by GitHub
commit f61040d0e1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -321,22 +321,13 @@ var _ = SIGDescribe("Load capacity", func() {
// to make it possible to create/schedule & delete them in the meantime.
// Currently we assume that <throughput> pods/second average throughput.
// The expected number of created/deleted pods is totalPods/4 when scaling
// for the first time, with each RC changing its size from X to a uniform
// random value in the interval [X/2, 3X/2].
// The expected number of created/deleted pods is totalPods/4 when scaling,
// as each RC changes its size from X to a uniform random value in [X/2, 3X/2].
scalingTime := time.Duration(totalPods/(4*throughput)) * time.Second
framework.Logf("Starting to scale %v objects first time...", itArg.kind)
scaleAllResources(configs, scalingTime, testPhaseDurations.StartPhase(300, "scaling first time"))
By("============================================================================")
// The expected number of created/deleted pods is totalPods/3 when scaling for
// the second time, with each RC changing its size from a uniform random value
// in the interval [X/2, 3X/2] to another uniform random value in same interval.
scalingTime = time.Duration(totalPods/(3*throughput)) * time.Second
framework.Logf("Starting to scale %v objects second time...", itArg.kind)
scaleAllResources(configs, scalingTime, testPhaseDurations.StartPhase(400, "scaling second time"))
By("============================================================================")
// Cleanup all created replication controllers.
// Currently we assume <throughput> pods/second average deletion throughput.
// We may want to revisit it in the future.