Merge pull request #1070 from vincepri/disable-leak-tests

Mark leak goroutine tests as pending

Author: k8s-ci-robot

pkg/controller/controller_test.go

@@ -93,7 +93,9 @@ var _ = Describe("controller.Controller", func() {
 			close(done)
 		})
 
-		It("should not leak goroutines when stop", func(done Done) {
+		// This test has been marked as pending because it has been causing lots of flakes in CI.
+		// It should be rewritten at some point later in the future.
+		XIt("should not leak goroutines when stop", func(done Done) {
 			// TODO(directxman12): After closing the proper leaks on watch this must be reduced to 0
 			// The leaks currently come from the event-related code (as in corev1.Event).
 			threshold := 3

pkg/manager/manager_test.go

@@ -1168,7 +1168,9 @@ var _ = Describe("manger.Manager", func() {
 		})
 	})
 
-	It("should not leak goroutines when stop", func(done Done) {
+	// This test has been marked as pending because it has been causing lots of flakes in CI.
+	// It should be rewritten at some point later in the future.
+	XIt("should not leak goroutines when stop", func(done Done) {
 		// TODO(directxman12): After closing the proper leaks on watch this must be reduced to 0
 		// The leaks currently come from the event-related code (as in corev1.Event).
 		threshold := 3