Commit 9aef074e authored by Bowei Du's avatar Bowei Du

Update the timeout in CLOSE_WAIT e2e test

I see some test flakes due to the timeout being too strict, updating to a larger value. Adding tail -n 1, it looks like there may be leftover state for other runs. We really only care about one of the CLOSE_WAIT entries.
parent 435268ac
...@@ -180,7 +180,8 @@ var _ = framework.KubeDescribe("Network", func() { ...@@ -180,7 +180,8 @@ var _ = framework.KubeDescribe("Network", func() {
fmt.Sprintf( fmt.Sprintf(
"sudo cat /proc/net/ip_conntrack "+ "sudo cat /proc/net/ip_conntrack "+
"| grep 'CLOSE_WAIT.*dst=%v.*dport=%v' "+ "| grep 'CLOSE_WAIT.*dst=%v.*dport=%v' "+
"| awk '{print $3}'", "| tail -n 1"+
"| awk '{print $3}' ",
serverNodeInfo.nodeIp, serverNodeInfo.nodeIp,
testDaemonTcpPort), testDaemonTcpPort),
framework.TestContext.Provider, framework.TestContext.Provider,
...@@ -193,7 +194,7 @@ var _ = framework.KubeDescribe("Network", func() { ...@@ -193,7 +194,7 @@ var _ = framework.KubeDescribe("Network", func() {
// These must be synchronized from the default values set in // These must be synchronized from the default values set in
// pkg/apis/../defaults.go ConntrackTCPCloseWaitTimeout. The // pkg/apis/../defaults.go ConntrackTCPCloseWaitTimeout. The
// current defaults are hidden in the initialization code. // current defaults are hidden in the initialization code.
const epsilonSeconds = 10 const epsilonSeconds = 60
const expectedTimeoutSeconds = 60 * 60 const expectedTimeoutSeconds = 60 * 60
framework.Logf("conntrack entry timeout was: %v, expected: %v", framework.Logf("conntrack entry timeout was: %v, expected: %v",
......
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