[master] 640b2e4 Don't wait for s2, as that should block forever on test success.

Martin Blix Grydeland martin at varnish-cache.org
Tue Mar 5 14:28:27 CET 2013


commit 640b2e4e10a82347927bee584aa258f62ccf2375
Author: Martin Blix Grydeland <martin at varnish-software.com>
Date:   Tue Mar 5 14:22:00 2013 +0100

    Don't wait for s2, as that should block forever on test success.
    
    Add a delay after setting parameters to allow acceptor thread to pick them up.
    
    Add a delay in c1 after receiving result to allow time for any fail causing
    fetches from s2

diff --git a/bin/varnishtest/tests.disabled/r01252.vtc b/bin/varnishtest/tests.disabled/r01252.vtc
index e055590..174ee97 100644
--- a/bin/varnishtest/tests.disabled/r01252.vtc
+++ b/bin/varnishtest/tests.disabled/r01252.vtc
@@ -2,7 +2,7 @@ varnishtest "#1252 - Drop remote closed connections returning from waitinglists"
 
 # This test case is disabled because it will only pass on platforms
 # where the tcp_keepalive_* runtime arguments are available, and also
-# because it requires "-t 75" argument to varnishtest (remote closed
+# because it requires "-t 80" argument to varnishtest (remote closed
 # state will only be detected after FIN timeout has passed (60s))
 
 server s1 {
@@ -39,11 +39,14 @@ varnish v1 -cliok "param.set tcp_keepalive_intvl 1s"
 varnish v1 -cliok "param.show first_byte_timeout"
 varnish v1 -cliok "param.set first_byte_timeout 70"
 
+delay 2
+
 client c1 {
 	timeout 70
 	txreq -hdr "X-Client: 1"
 	rxresp
 	expect resp.status == 503
+	delay 2
 } -start
 
 client c2 {
@@ -54,4 +57,3 @@ client c2 {
 
 client c1 -wait
 client c2 -wait
-server s2 -wait



More information about the varnish-commit mailing list