Merge pull request kubernetes#87057 from oomichi/add-debugging-msg-issue86678

k8s-ci-robot · web-flow · commit 460fdc7f488e · 2020-02-12T05:22:51.000-08:00
Add logs of port-forward-tester pod
diff --git a/test/e2e/kubectl/portforward.go b/test/e2e/kubectl/portforward.go
@@ -333,6 +333,12 @@ func doTestMustConnectSendDisconnect(bindAddress string, f *framework.Framework)
 	}
 
 	if e, a := strings.Repeat("x", 100), string(fromServer); e != a {
+		podlogs, err := e2epod.GetPodLogs(f.ClientSet, f.Namespace.Name, pod.Name, "portforwardtester")
+		if err != nil {
+			framework.Logf("Failed to get logs of portforwardtester pod: %v", err)
+		} else {
+			framework.Logf("Logs of portforwardtester pod: %v", podlogs)
+		}
 		framework.Failf("Expected %q from server, got %q", e, a)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -333,6 +333,12 @@ func doTestMustConnectSendDisconnect(bindAddress string, f *framework.Framework)`
`333`	`333`	`}`
`334`	`334`
`335`	`335`	`if e, a := strings.Repeat("x", 100), string(fromServer); e != a {`
	`336`	`+ podlogs, err := e2epod.GetPodLogs(f.ClientSet, f.Namespace.Name, pod.Name, "portforwardtester")`
	`337`	`+ if err != nil {`
	`338`	`+ framework.Logf("Failed to get logs of portforwardtester pod: %v", err)`
	`339`	`+ } else {`
	`340`	`+ framework.Logf("Logs of portforwardtester pod: %v", podlogs)`
	`341`	`+ }`
`336`	`342`	`framework.Failf("Expected %q from server, got %q", e, a)`
`337`	`343`	`}`
`338`	`344`