Skip to content

K
k3s
Commit 4640b7e2 authored by Davanum Srinivas's avatar Davanum Srinivas
Browse files
Options

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.
parent 0a0b6889
Hide whitespace changes
Inline Side-by-side
Showing with 1 addition and 1 deletion
test/e2e/dns.go
View file @ 4640b7e2
...@@ -321,10 +321,10 @@ var _ = framework.KubeDescribe("DNS", func() { ...@@ -321,10 +321,10 @@ var _ = framework.KubeDescribe("DNS", func() {
"kubernetes.default", "kubernetes.default",
"kubernetes.default.svc", "kubernetes.default.svc",
"kubernetes.default.svc.cluster.local", "kubernetes.default.svc.cluster.local",
"google.com",
} }
// Added due to #8512. This is critical for GCE and GKE deployments. // Added due to #8512. This is critical for GCE and GKE deployments.
if framework.ProviderIs("gce", "gke") { if framework.ProviderIs("gce", "gke") {
namesToResolve = append(namesToResolve, "google.com")
namesToResolve = append(namesToResolve, "metadata") namesToResolve = append(namesToResolve, "metadata")
} }
hostFQDN := fmt.Sprintf("%s.%s.%s.svc.cluster.local", dnsTestPodHostName, dnsTestServiceName, f.Namespace.Name) hostFQDN := fmt.Sprintf("%s.%s.%s.svc.cluster.local", dnsTestPodHostName, dnsTestServiceName, f.Namespace.Name)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment