From 4316ce278fb4ea62336e3592336812ff304d9fc4 Mon Sep 17 00:00:00 2001 From: Brendan Burns Date: Sun, 23 Oct 2016 22:21:35 -0700 Subject: [PATCH] Don't count failed pods as "not-ready" --- test/e2e/cluster_size_autoscaling.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/test/e2e/cluster_size_autoscaling.go b/test/e2e/cluster_size_autoscaling.go index 58136082ebf..5f1f0381627 100644 --- a/test/e2e/cluster_size_autoscaling.go +++ b/test/e2e/cluster_size_autoscaling.go @@ -565,7 +565,12 @@ func waitForAllCaPodsReadyInNamespace(f *framework.Framework, c *client.Client) ready = true } } - if !ready { + // Failed pods in this context generally mean that they have been + // double scheduled onto a node, but then failed a constraint check. + if pod.Status.Phase == api.PodFailed { + glog.Warningf("Pod has failed: %v", pod) + } + if !ready && pod.Status.Phase != api.PodFailed { notready = append(notready, pod.Name) } }