From 72be89523831d945e37ccf09a336f806e4e2077c Mon Sep 17 00:00:00 2001 From: Claudiu Belu Date: Fri, 11 Jan 2019 03:53:28 -0800 Subject: [PATCH] tests: Makes UDP testing command more Windows friendly By default, cmd is used to execute commands in Windows containers, but it has some issues executing the command (no output): echo 'hostName' | nc -w 1 -u But it has no issues running (output is correct): echo hostName | nc -w 1 -u This causes the test "should function for node-pod communication: udp" to fail when using Windows Containers. --- test/e2e/framework/networking_utils.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/e2e/framework/networking_utils.go b/test/e2e/framework/networking_utils.go index a8f8831d2c..01de115692 100644 --- a/test/e2e/framework/networking_utils.go +++ b/test/e2e/framework/networking_utils.go @@ -292,7 +292,7 @@ func (config *NetworkingTestConfig) DialFromNode(protocol, targetIP string, targ if protocol == "udp" { // TODO: It would be enough to pass 1s+epsilon to timeout, but unfortunately // busybox timeout doesn't support non-integer values. - cmd = fmt.Sprintf("echo 'hostName' | nc -w 1 -u %s %d", targetIP, targetPort) + cmd = fmt.Sprintf("echo hostName | nc -w 1 -u %s %d", targetIP, targetPort) } else { ipPort := net.JoinHostPort(targetIP, strconv.Itoa(targetPort)) // The current versions of curl included in CentOS and RHEL distros