From 3a0b253f86f353ab7d1b0723e0ee633f610cf73b Mon Sep 17 00:00:00 2001 From: Jean-Christophe Sirot Date: Tue, 4 Sep 2018 12:02:35 +0200 Subject: [PATCH 1/2] Fix the skip message since it was the exact opposite of the tested condition Signed-off-by: Jean-Christophe Sirot --- test/e2e/framework/util.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/e2e/framework/util.go b/test/e2e/framework/util.go index 2b8786459c2..046c26d5927 100644 --- a/test/e2e/framework/util.go +++ b/test/e2e/framework/util.go @@ -359,7 +359,7 @@ func SkipIfMultizone(c clientset.Interface) { Skipf("Error listing cluster zones") } if zones.Len() > 1 { - Skipf("Requires more than one zone") + Skipf("Requires exactly one zone") } } From 6900259480e6fe02010729e7a87ca28cb305fd1d Mon Sep 17 00:00:00 2001 From: Jean-Christophe Sirot Date: Tue, 4 Sep 2018 15:10:18 +0200 Subject: [PATCH 2/2] Update the message to be more precise regarding the tested condition Signed-off-by: Jean-Christophe Sirot --- test/e2e/framework/util.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/e2e/framework/util.go b/test/e2e/framework/util.go index 046c26d5927..f7db583d8b9 100644 --- a/test/e2e/framework/util.go +++ b/test/e2e/framework/util.go @@ -359,7 +359,7 @@ func SkipIfMultizone(c clientset.Interface) { Skipf("Error listing cluster zones") } if zones.Len() > 1 { - Skipf("Requires exactly one zone") + Skipf("Requires at most one zone") } }