github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-30 06:54:01 +00:00
Code Issues Projects Releases Wiki Activity

check externalNameService more time

Browse Source
This commit is contained in:
jornshen 2021-01-29 11:28:06 +08:00
parent 228d5f2002
commit 415fe230b4
1 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
test/e2e/framework/service/jig.go
View File

@ -988,13 +988,16 @@ func (j *TestJig) checkExternalServiceReachability(svc *v1.Service, pod *v1.Pod)
svcName := fmt.Sprintf("%s.%s.svc.%s", svc.Name, svc.Namespace, framework.TestContext.ClusterDNSDomain) svcName := fmt.Sprintf("%s.%s.svc.%s", svc.Name, svc.Namespace, framework.TestContext.ClusterDNSDomain)
// Service must resolve to IP // Service must resolve to IP
cmd := fmt.Sprintf("nslookup %s", svcName) cmd := fmt.Sprintf("nslookup %s", svcName)
_, stderr, err := framework.RunHostCmdWithFullOutput(pod.Namespace, pod.Name, cmd) return wait.PollImmediate(framework.Poll, ServiceReachabilityShortPollTimeout, func() (done bool, err error) {
// NOTE(claudiub): nslookup may return 0 on Windows, even though the DNS name was not found. In this case, _, stderr, err := framework.RunHostCmdWithFullOutput(pod.Namespace, pod.Name, cmd)
// we can check stderr for the error. // NOTE(claudiub): nslookup may return 0 on Windows, even though the DNS name was not found. In this case,
if err != nil || (framework.NodeOSDistroIs("windows") && strings.Contains(stderr, fmt.Sprintf("can't find %s", svcName))) { // we can check stderr for the error.
return fmt.Errorf("ExternalName service %q must resolve to IP", pod.Namespace+"/"+pod.Name) if err != nil || (framework.NodeOSDistroIs("windows") && strings.Contains(stderr, fmt.Sprintf("can't find %s", svcName))) {
} framework.Logf("ExternalName service %q failed to resolve to IP", pod.Namespace+"/"+pod.Name)
return nil return false, nil
}
return true, nil
})
} }
// CheckServiceReachability ensures that request are served by the services. Only supports Services with type ClusterIP, NodePort and ExternalName. // CheckServiceReachability ensures that request are served by the services. Only supports Services with type ClusterIP, NodePort and ExternalName.