Merge pull request kubernetes#5387 from rrati/density-cleanup-fix-5385

Fixed issue with the density test failing after a successful run because...
duda · Mar 12, 2015 · 6a0bfd7 · 6a0bfd7
2 parents 55ed70f + 3c9a9a4
commit 6a0bfd7
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/test/e2e/density.go b/test/e2e/density.go
@@ -187,8 +187,14 @@ var _ = PDescribe("Density", func() {
 	})
 
 	AfterEach(func() {
-		// Remove any remaining pods from this test
-		DeleteRC(c, ns, RCName)
+		// Remove any remaining pods from this test if the
+		// replication controller still exists and the replica count
+		// isn't 0.  This means the controller wasn't cleaned up
+		// during the test so clean it up here
+		rc, err := c.ReplicationControllers(ns).Get(RCName)
+		if err == nil && rc.Spec.Replicas != 0 {
+			DeleteRC(c, ns, RCName)
+		}
 	})
 
 	It("should allow starting 100 pods per node", func() {