[master] 2b94cd5 v1 actually forwards traffic for v2

Dridi Boukelmoune dridi.boukelmoune at gmail.com
Mon Mar 6 17:01:05 CET 2017


commit 2b94cd5e70cef26414400248fcc5c1b0f83d1b34
Author: Dridi Boukelmoune <dridi.boukelmoune at gmail.com>
Date:   Mon Mar 6 16:58:26 2017 +0100

    v1 actually forwards traffic for v2
    
    Remains of copy/pasta made that v2 listened with PROXY protocol which is
    not needed. It's a passive instance that only does health checking using
    a probe.

diff --git a/bin/varnishtest/tests/o00004.vtc b/bin/varnishtest/tests/o00004.vtc
index c9fb5d4..f51bcc6 100644
--- a/bin/varnishtest/tests/o00004.vtc
+++ b/bin/varnishtest/tests/o00004.vtc
@@ -2,16 +2,16 @@ varnishtest "Sending proxy headers via health probes"
 
 # Double-proxy scheme stolen from o00002.vtc
 
-# Get ${v1_addr} defined so s1 can use it so v1 can use ${s1_addr}
-varnish v1 -proto PROXY -vcl {backend foo { .host = ":80"; }} -start
+# Get ${v2_addr} defined so s1 can use it
+varnish v2 -arg "-b '${bad_backend}'" -start
 
 server s1 {
 	rxreq
-	expect req.http.x-forwarded-for == ${v1_addr}
+	expect req.http.x-forwarded-for == ${v2_addr}
 	txresp
 } -start
 
-varnish v1 -vcl+backend {
+varnish v1 -proto PROXY -vcl+backend {
 	import debug;
 
 	sub vcl_recv {
@@ -19,9 +19,9 @@ varnish v1 -vcl+backend {
 			return (synth(400));
 		}
 	}
-}
+} -start
 
-varnish v2 -proto PROXY -vcl {
+varnish v2 -vcl {
 	import std;
 
 	probe default {
@@ -44,7 +44,7 @@ varnish v2 -proto PROXY -vcl {
 		# dummy backend ref
 		if (bp2) { }
 	}
-} -start
+}
 
 server s1 -wait
 



More information about the varnish-commit mailing list