From 7063da41daa6507819b176423bbd85c03333c893 Mon Sep 17 00:00:00 2001 From: jayunit100 Date: Fri, 22 May 2015 10:04:09 -0400 Subject: [PATCH] Make minStartupPods feature backward compatible default value = 0. --- test/e2e/e2e_test.go | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/test/e2e/e2e_test.go b/test/e2e/e2e_test.go index 75b36b5c82..1fa1e5a606 100644 --- a/test/e2e/e2e_test.go +++ b/test/e2e/e2e_test.go @@ -42,7 +42,9 @@ const ( podStartupTimeout = 10 * time.Minute // minStartupPods is the minimum number of pods that will allow - // wiatForPodsRunningReady(...) to succeed. More verbosely, that function + // waitForPodsRunningReady(...) to succeed (i.e. WLOG if you know that + // "DNS", and "Prometheus" pods need to be running, you might set it to "2"). + // More verbosely, that function // checks that all pods in the cluster are both in a phase of "running" and // have a condition of "ready": "true". It aims to ensure that the cluster's // pods are fully healthy before beginning e2e tests. However, if there were @@ -54,7 +56,8 @@ const ( // does *not* mean that the function will succeed as soon as minStartupPods // are found to be running and ready; it ensures that *all* pods it finds // are running and ready. This is the minimum number it must find. - minStartupPods = 1 + // TODO : Add command line option for this so that the number is non trivial. + minStartupPods = 0 ) var (