From 4640b7e264ca1417cb02e787e6263db9aa3dfcb7 Mon Sep 17 00:00:00 2001 From: Davanum Srinivas Date: Sat, 29 Apr 2017 10:35:17 -0400 Subject: [PATCH] Add google.com to e2e test only under gce/gke We should limit the lookup/resolve for google.com when provider is gce or gke. We should be able to run the test in environments where this is not allowed or not available. --- test/e2e/dns.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/e2e/dns.go b/test/e2e/dns.go index eca2386cf90..dc3bdf32868 100644 --- a/test/e2e/dns.go +++ b/test/e2e/dns.go @@ -321,10 +321,10 @@ var _ = framework.KubeDescribe("DNS", func() { "kubernetes.default", "kubernetes.default.svc", "kubernetes.default.svc.cluster.local", - "google.com", } // Added due to #8512. This is critical for GCE and GKE deployments. if framework.ProviderIs("gce", "gke") { + namesToResolve = append(namesToResolve, "google.com") namesToResolve = append(namesToResolve, "metadata") } hostFQDN := fmt.Sprintf("%s.%s.%s.svc.cluster.local", dnsTestPodHostName, dnsTestServiceName, f.Namespace.Name)