[master] e3c2808ab http2/vtc: Stabilize WR_SHUT test

Nils Goroll nils.goroll at uplex.de
Mon Mar 3 20:22:05 UTC 2025


commit e3c2808abf3a957c7c699b91fe4f9c046f297483
Author: Nils Goroll <nils.goroll at uplex.de>
Date:   Mon Mar 3 21:18:31 2025 +0100

    http2/vtc: Stabilize WR_SHUT test
    
    If a shutdown command is last in the vtc client, it closes the file descriptor
    right after the shutdown, such that varnishd can not reliably send the GOAWAY,
    which in turn changes the H2 error message.
    
    Ref #4283
    Ref #4284
    Closes #4293

diff --git a/bin/varnishtest/tests/t02027.vtc b/bin/varnishtest/tests/t02027.vtc
index 02bad0f7c..5bc7b4816 100644
--- a/bin/varnishtest/tests/t02027.vtc
+++ b/bin/varnishtest/tests/t02027.vtc
@@ -22,6 +22,12 @@ logexpect l0 -v v1 -g vxid -q "Begin ~ sess" {
 client c0 {
 	txpri
 	shutdown -write
+        stream 0 {
+		rxsettings
+                rxgoaway
+                expect goaway.laststream == 0
+                expect goaway.err == NO_ERROR
+	} -run
 } -run
 
 logexpect l1 -v v1 -g vxid -q "Begin ~ sess" {
@@ -40,6 +46,11 @@ client c1 {
 		txreq -nohdrend
 	} -run
 	shutdown -write
+        stream 0 {
+                rxgoaway
+                expect goaway.laststream == 1
+                expect goaway.err == NO_ERROR
+	} -run
 } -run
 
 logexpect l2 -v v1 -g vxid -q "Begin ~ sess" {
@@ -58,6 +69,11 @@ client c2 {
 		txreq -nostrend
 	} -run
 	shutdown -write
+        stream 0 {
+                rxgoaway
+                expect goaway.laststream == 1
+                expect goaway.err == NO_ERROR
+	} -run
 } -run
 
 logexpect l3 -v v1 -g vxid -q "Begin ~ sess" {
@@ -85,6 +101,11 @@ client c3 {
 		#	00053a70617468012f00073a6d6574686f640347455400073a736368656d650468747470
 	} -run
 	shutdown -write
+        stream 0 {
+                rxgoaway
+                expect goaway.laststream == 0
+                expect goaway.err == NO_ERROR
+	} -run
 } -run
 
 logexpect l0 -wait


More information about the varnish-commit mailing list