From 1631c6dd247b83e8f92d075fb810ededac66a67e Mon Sep 17 00:00:00 2001 From: Jeff Lowdermilk Date: Tue, 13 Sep 2016 19:28:46 -0700 Subject: [PATCH] update comment, only skip on missing resource if provider is not gce --- test/e2e/petset.go | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/test/e2e/petset.go b/test/e2e/petset.go index 926447ddf0c..0818a1bc669 100644 --- a/test/e2e/petset.go +++ b/test/e2e/petset.go @@ -67,10 +67,13 @@ var _ = framework.KubeDescribe("PetSet [Slow] [Feature:PetSet]", func() { var c *client.Client BeforeEach(func() { - // PetSet is in alpha, so it's disabled on all platforms except GCE. + // PetSet is in alpha, so it's disabled on some platforms. We skip this + // test if a resource get fails on non-GCE platforms. // In theory, tests that restart pets should pass on any platform with a // dynamic volume provisioner. - framework.SkipIfMissingResource(f.ClientPool, unversioned.GroupVersionResource{Group: apps.GroupName, Version: "v1alpha1", Resource: "petsets"}, f.Namespace.Name) + if !framework.ProviderIs("gce") { + framework.SkipIfMissingResource(f.ClientPool, unversioned.GroupVersionResource{Group: apps.GroupName, Version: "v1alpha1", Resource: "petsets"}, f.Namespace.Name) + } c = f.Client ns = f.Namespace.Name