Skip to content

Commit

Permalink
Fixed issue with the density test failing after a successful run beca…
Browse files Browse the repository at this point in the history
…use of

a failure to cleanup kubernetes#5385
  • Loading branch information
Robert Rati committed Mar 12, 2015
1 parent c03b080 commit 3c9a9a4
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions test/e2e/density.go
Original file line number Diff line number Diff line change
Expand Up @@ -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() {
Expand Down

0 comments on commit 3c9a9a4

Please sign in to comment.