Fix bug with profile-gathering waitgroup in scale tests

db0a3721 · Shyam Jeedigunta · b5387eff · db0a3721 · db0a3721
Unverified Commit db0a3721 authored Feb 08, 2018 by Shyam Jeedigunta
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 0 deletions

density.go test/e2e/scalability/density.go +1 -0

load.go test/e2e/scalability/load.go +1 -0

No files found.
--- a/test/e2e/scalability/density.go
+++ b/test/e2e/scalability/density.go
@@ -363,6 +363,7 @@ var _ = SIGDescribe("Density", func() {
 		// Stop apiserver CPU profile gatherer and gather memory allocations profile.
 		close(profileGathererStopCh)
 		wg := sync.WaitGroup{}
+		wg.Add(1)
 		framework.GatherApiserverMemoryProfile(&wg, "density")
 		wg.Wait()


--- a/test/e2e/scalability/load.go
+++ b/test/e2e/scalability/load.go
@@ -104,6 +104,7 @@ var _ = SIGDescribe("Load capacity", func() {
 		// Stop apiserver CPU profile gatherer and gather memory allocations profile.
 		close(profileGathererStopCh)
 		wg := sync.WaitGroup{}
+		wg.Add(1)
 		framework.GatherApiserverMemoryProfile(&wg, "load")
 		wg.Wait()