You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2022/08/30 23:51:18 UTC

[trafficserver] branch 9.2.x updated: Disable the QUIC TestRun in active_timeout AuTest (#9069)

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 9.2.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/9.2.x by this push:
     new a68515712 Disable the QUIC TestRun in active_timeout AuTest (#9069)
a68515712 is described below

commit a68515712111afcebc3df973150a0b373fd90ddd
Author: Brian Neradt <br...@gmail.com>
AuthorDate: Mon Aug 29 18:15:11 2022 -0500

    Disable the QUIC TestRun in active_timeout AuTest (#9069)
    
    We have added QUIC version 0x1 support to our 10-Dev branch via #8956,
    but that has not been backported to 9.x and we don't plan to backport
    that at this time. ASF CI will be updated with the latest version of
    curl which uses 0x1 and would therefore fail. This change disables the
    QUIC TestRun of active_timeout so it doesn't run on 9.x since ATS
    doesn't support the latest QUIC test tools on that branch. This patch,
    therefore, should not go into 10-Dev.
    
    Note that this patch can be reverted in the future if we decide to
    backport QUIC 0x1 support to 9.x.
    
    (cherry picked from commit 98cc66b323d2975120d5db4a39bd363054bd8618)
---
 tests/gold_tests/timeout/active_timeout.test.py | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/tests/gold_tests/timeout/active_timeout.test.py b/tests/gold_tests/timeout/active_timeout.test.py
index e43b68eaa..dcf999816 100644
--- a/tests/gold_tests/timeout/active_timeout.test.py
+++ b/tests/gold_tests/timeout/active_timeout.test.py
@@ -64,7 +64,10 @@ tr3 = Test.AddTestRun("tr")
 tr3.Processes.Default.Command = 'curl -k -i --http2 https://127.0.0.1:{0}/file'.format(ts.Variables.ssl_port)
 tr3.Processes.Default.Streams.stdout = Testers.ContainsExpression("Activity Timeout", "Request should fail with active timeout")
 
-if Condition.HasATSFeature('TS_USE_QUIC') and Condition.HasCurlFeature('http3'):
-    tr4 = Test.AddTestRun("tr")
-    tr4.Processes.Default.Command = 'curl -k -i --http3 https://127.0.0.1:{0}/file'.format(ts.Variables.ssl_port)
-    tr4.Processes.Default.Streams.stdout = Testers.ContainsExpression("Activity Timeout", "Request should fail with active timeout")
+# Commenting out the HTTP/3 test since 9.x and before does not support the
+# latest version of HTTP/3 which is used by curl. ATS 10.x does support this
+# later version, so this test is run for that release and later.
+# if Condition.HasATSFeature('TS_USE_QUIC') and Condition.HasCurlFeature('http3'):
+#     tr4 = Test.AddTestRun("tr")
+#     tr4.Processes.Default.Command = 'curl -k -i --http3 https://127.0.0.1:{0}/file'.format(ts.Variables.ssl_port)
+#     tr4.Processes.Default.Streams.stdout = Testers.ContainsExpression("Activity Timeout", "Request should fail with active timeout")