Merge pull request #31516 from sttts/sttts-e2e-CheckConnectivityToHost-logs

Automatic merge from submit-queue e2e: log wget output on CheckConnectivityToHost error Log output might help to diagnose e2e flakes, whether they are caused by dns issues or connection timeouts. Might help with flake https://github.com/kubernetes/kubernetes/issues/28188.

Merge pull request #31516 from sttts/sttts-e2e-CheckConnectivityToHost-logs
0f8869d3 · Kubernetes Submit Queue · GitHub · be848f2a · 3262b7fc · 0f8869d3
Commit 0f8869d3 authored Aug 30, 2016 by Kubernetes Submit Queue Committed by GitHub Aug 30, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 12 additions and 1 deletion

util.go test/e2e/framework/util.go +12 -1

No files found.
--- a/test/e2e/framework/util.go
+++ b/test/e2e/framework/util.go
@@ -4689,7 +4689,18 @@ func CheckConnectivityToHost(f *Framework, nodeName, podName, host string, timeo
 		return err
 	}
 	defer podClient.Delete(podName, nil)
-	return WaitForPodSuccessInNamespace(f.Client, podName, contName, f.Namespace.Name)
+	err = WaitForPodSuccessInNamespace(f.Client, podName, contName, f.Namespace.Name)
+
+	if err != nil {
+		logs, logErr := GetPodLogs(f.Client, f.Namespace.Name, pod.Name, contName)
+		if logErr != nil {
+			Logf("Warning: Failed to get logs from pod %q: %v", pod.Name, logErr)
+		} else {
+			Logf("pod %s/%s \"wget\" logs:\n%s", f.Namespace.Name, pod.Name, logs)
+		}
+	}
+
+	return err
 }

 // CoreDump SSHs to the master and all nodes and dumps their logs into dir.