Merge pull request #21796 from nikhiljindal/deploymente2eLogs

deployment e2e test flake: Print the remaining pods to debug test flake
This commit is contained in:
Dawn Chen 2016-02-23 16:43:00 -08:00
commit b19102ba23

View File

@ -183,8 +183,9 @@ func stopDeployment(c *clientset.Clientset, oldC client.Interface, ns, deploymen
Expect(err).NotTo(HaveOccurred()) Expect(err).NotTo(HaveOccurred())
Expect(rss.Items).Should(HaveLen(0)) Expect(rss.Items).Should(HaveLen(0))
Logf("ensuring deployment %s pods were deleted", deploymentName) Logf("ensuring deployment %s pods were deleted", deploymentName)
var pods *api.PodList
if err := wait.PollImmediate(time.Second, wait.ForeverTestTimeout, func() (bool, error) { if err := wait.PollImmediate(time.Second, wait.ForeverTestTimeout, func() (bool, error) {
pods, err := c.Core().Pods(ns).List(api.ListOptions{}) pods, err = c.Core().Pods(ns).List(api.ListOptions{})
if err != nil { if err != nil {
return false, err return false, err
} }
@ -193,7 +194,7 @@ func stopDeployment(c *clientset.Clientset, oldC client.Interface, ns, deploymen
} }
return false, nil return false, nil
}); err != nil { }); err != nil {
Failf("Failed to remove deployment %s pods!", deploymentName) Failf("Err : %s\n. Failed to remove deployment %s pods : %+v", deploymentName, pods)
} }
} }