Merge pull request #41420 from jbeda/add-public-to-e2e

Automatic merge from submit-queue Adds kube-public to the whitelist to not be deleted for e2e tests We added the `kube-public` namespace but didn't add it to a whitelist of namespaces to not delete as part of e2e cleanup. ```release-note ```

Merge pull request #41420 from jbeda/add-public-to-e2e
bd1a2221 · Kubernetes Submit Queue · GitHub · 00b02117 · 93752947 · bd1a2221
Commit bd1a2221 authored Feb 19, 2017 by Kubernetes Submit Queue Committed by GitHub Feb 19, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 2 deletions

e2e.go test/e2e/e2e.go +8 -2

No files found.
--- a/test/e2e/e2e.go
+++ b/test/e2e/e2e.go
@@ -108,10 +108,16 @@ var _ = ginkgo.SynchronizedBeforeSuite(func() []byte {
 		glog.Fatal("Error loading client: ", err)
 	}
-	// Delete any namespaces except default and kube-system. This ensures no
+	// Delete any namespaces except those created by the system. This ensures no
 	// lingering resources are left over from a previous test run.
 	if framework.TestContext.CleanStart {
-		deleted, err := framework.DeleteNamespaces(c, nil /* deleteFilter */, []string{metav1.NamespaceSystem, metav1.NamespaceDefault, federationapi.FederationNamespaceSystem})
+		deleted, err := framework.DeleteNamespaces(c, nil, /* deleteFilter */
+			[]string{
+				metav1.NamespaceSystem,
+				metav1.NamespaceDefault,
+				metav1.NamespacePublic,
+				federationapi.FederationNamespaceSystem,
+			})
 		if err != nil {
 			framework.Failf("Error deleting orphaned namespaces: %v", err)
 		}