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 2021/03/29 19:25:20 UTC

[trafficserver] branch 9.0.x updated: Increase Proxy Verifier caching delay. (#7616)

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

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


The following commit(s) were added to refs/heads/9.0.x by this push:
     new 1a0808b  Increase Proxy Verifier caching delay. (#7616)
1a0808b is described below

commit 1a0808b43ec891a0044aecb4b9994a799acce4ad
Author: Brian Neradt <br...@verizonmedia.com>
AuthorDate: Tue Mar 23 09:44:05 2021 -0500

    Increase Proxy Verifier caching delay. (#7616)
    
    Increasing the Proxy Verifier caching delay from 50ms to 100ms to
    accommodate slow IO situations.
    
    (cherry picked from commit 3d266379b816227928171d1f07de747f6a9a254b)
---
 .../cache/replay/cache-control-max-age.replay.yaml         | 14 +++++++-------
 .../replay/negative-caching-300-second-timeout.replay.yaml |  2 +-
 .../cache/replay/negative-caching-customized.replay.yaml   |  6 +++---
 .../cache/replay/negative-caching-default.replay.yaml      |  8 ++++----
 .../cache/replay/negative-caching-disabled.replay.yaml     |  8 ++++----
 .../cache/replay/negative-caching-no-timeout.replay.yaml   |  2 +-
 .../cache/replay/negative-caching-timeout.replay.yaml      |  2 +-
 7 files changed, 21 insertions(+), 21 deletions(-)

diff --git a/tests/gold_tests/cache/replay/cache-control-max-age.replay.yaml b/tests/gold_tests/cache/replay/cache-control-max-age.replay.yaml
index 8e88e71..d97bf15 100644
--- a/tests/gold_tests/cache/replay/cache-control-max-age.replay.yaml
+++ b/tests/gold_tests/cache/replay/cache-control-max-age.replay.yaml
@@ -36,7 +36,7 @@ meta:
 
         # Add a delay so ATS has time to finish any caching IO for the previous
         # transaction.
-        delay: 50ms
+        delay: 100ms
 
   - request_for_zero_max_age: &request_for_zero_max_age
       client-request:
@@ -50,7 +50,7 @@ meta:
 
         # Add a delay so ATS has time to finish any caching IO for the previous
         # transaction.
-        delay: 50ms
+        delay: 100ms
 
   - request_for_negative_max_age: &request_for_negative_max_age
       client-request:
@@ -64,7 +64,7 @@ meta:
 
         # Add a delay so ATS has time to finish any caching IO for the previous
         # transaction.
-        delay: 50ms
+        delay: 100ms
 
   - request_for_non_number_max_age: &request_for_non_number_max_age
       client-request:
@@ -78,7 +78,7 @@ meta:
 
         # Add a delay so ATS has time to finish any caching IO for the previous
         # transaction.
-        delay: 50ms
+        delay: 100ms
 
   - 200_ok_response: &200_ok_response
       server-response:
@@ -267,7 +267,7 @@ sessions:
 
       # Add a delay so ATS has time to finish any caching IO for the previous
       # transaction.
-      delay: 50ms
+      delay: 100ms
 
     # This should be replied to out of the cache, therefore this 400 response
     # should not make it to the client.
@@ -318,7 +318,7 @@ sessions:
 
       # Add a delay so ATS has time to finish any caching IO for the previous
       # transaction.
-      delay: 50ms
+      delay: 100ms
 
     # This should go through to the server because the 0 max-age
     # directive in the request should make ATS consider it stale.
@@ -371,7 +371,7 @@ sessions:
 
       # Add a delay so ATS has time to finish any caching IO for the previous
       # transaction.
-      delay: 50ms
+      delay: 100ms
 
     # This should go through to the server because the negative max-age
     # directive in the request should make ATS consider it stale.
diff --git a/tests/gold_tests/cache/replay/negative-caching-300-second-timeout.replay.yaml b/tests/gold_tests/cache/replay/negative-caching-300-second-timeout.replay.yaml
index 536c7a6..7cb667b 100644
--- a/tests/gold_tests/cache/replay/negative-caching-300-second-timeout.replay.yaml
+++ b/tests/gold_tests/cache/replay/negative-caching-300-second-timeout.replay.yaml
@@ -38,7 +38,7 @@ meta:
 
         # Add a delay so ATS has time to finish any caching IO for the previous
         # transaction.
-        delay: 50ms
+        delay: 100ms
 
 sessions:
 - transactions:
diff --git a/tests/gold_tests/cache/replay/negative-caching-customized.replay.yaml b/tests/gold_tests/cache/replay/negative-caching-customized.replay.yaml
index 3b12d46..52df653 100644
--- a/tests/gold_tests/cache/replay/negative-caching-customized.replay.yaml
+++ b/tests/gold_tests/cache/replay/negative-caching-customized.replay.yaml
@@ -74,7 +74,7 @@ sessions:
 
       # Add a delay so ATS has time to finish any caching IO for the previous
       # transaction.
-      delay: 50ms
+      delay: 100ms
 
     # Since 404 responses are customized to not be cached, this will go
     # through.
@@ -121,7 +121,7 @@ sessions:
 
       # Add a delay so ATS has time to finish any caching IO for the previous
       # transaction.
-      delay: 50ms
+      delay: 100ms
 
     # By customization, the 400 will be cached and this will not go through.
     <<: *200_response
@@ -161,7 +161,7 @@ sessions:
 
       # Add a delay so ATS has time to finish any caching IO for the previous
       # transaction.
-      delay: 50ms
+      delay: 100ms
 
     # This should not go to the server since the 200 response is cached.
     server-response:
diff --git a/tests/gold_tests/cache/replay/negative-caching-default.replay.yaml b/tests/gold_tests/cache/replay/negative-caching-default.replay.yaml
index 0ab66f5..6b945c3 100644
--- a/tests/gold_tests/cache/replay/negative-caching-default.replay.yaml
+++ b/tests/gold_tests/cache/replay/negative-caching-default.replay.yaml
@@ -71,7 +71,7 @@ sessions:
 
       # Add a delay so ATS has time to finish any caching IO for the previous
       # transaction.
-      delay: 50ms
+      delay: 100ms
 
     # 404 responses should be cached when negative caching is enabled, so this
     # should not get to the server.  But if it does, return a 200 so the test
@@ -119,7 +119,7 @@ sessions:
 
       # Add a delay so ATS has time to finish any caching IO for the previous
       # transaction.
-      delay: 50ms
+      delay: 100ms
 
     # 400 responses should not be cached. Verify this goes to the server
     # by returning and expecting a 200 response.
@@ -159,7 +159,7 @@ sessions:
 
       # Add a delay so ATS has time to finish any caching IO for the previous
       # transaction.
-      delay: 50ms
+      delay: 100ms
 
     # This should not go to the server, but if it does return a 400 so we catch
     # it.
@@ -211,7 +211,7 @@ sessions:
 
       # Add a delay so ATS has time to finish any caching IO for the previous
       # transaction.
-      delay: 50ms
+      delay: 100ms
 
     # 405 responses should not be cached. Verify this goes to the server
     # by returning and expecting a 200 response.
diff --git a/tests/gold_tests/cache/replay/negative-caching-disabled.replay.yaml b/tests/gold_tests/cache/replay/negative-caching-disabled.replay.yaml
index d09362a..0df4f31 100644
--- a/tests/gold_tests/cache/replay/negative-caching-disabled.replay.yaml
+++ b/tests/gold_tests/cache/replay/negative-caching-disabled.replay.yaml
@@ -34,7 +34,7 @@ meta:
 
         # Add a delay so ATS has time to finish any caching IO for the previous
         # transaction.
-        delay: 50ms
+        delay: 100ms
 
   - request_for_path_404: &request_for_path_404
       client-request:
@@ -48,7 +48,7 @@ meta:
 
         # Add a delay so ATS has time to finish any caching IO for the previous
         # transaction.
-        delay: 50ms
+        delay: 100ms
 
   - request_for_no_cache_control_response: &request_for_no_cache_control_response
       client-request:
@@ -62,7 +62,7 @@ meta:
 
         # Add a delay so ATS has time to finish any caching IO for the previous
         # transaction.
-        delay: 50ms
+        delay: 100ms
 
 
   - request_for_404_with_cc: &request_for_404_with_cc
@@ -77,7 +77,7 @@ meta:
 
         # Add a delay so ATS has time to finish any caching IO for the previous
         # transaction.
-        delay: 50ms
+        delay: 100ms
 
 sessions:
 - transactions:
diff --git a/tests/gold_tests/cache/replay/negative-caching-no-timeout.replay.yaml b/tests/gold_tests/cache/replay/negative-caching-no-timeout.replay.yaml
index 634b141..358e699 100644
--- a/tests/gold_tests/cache/replay/negative-caching-no-timeout.replay.yaml
+++ b/tests/gold_tests/cache/replay/negative-caching-no-timeout.replay.yaml
@@ -41,7 +41,7 @@ sessions:
 
       # Add a delay so ATS has time to finish any caching IO for the previous
       # transaction.
-      delay: 50ms
+      delay: 100ms
 
     # This should not go to the server. Verify we get the cached 404 response
     # instead of this new 403 response.
diff --git a/tests/gold_tests/cache/replay/negative-caching-timeout.replay.yaml b/tests/gold_tests/cache/replay/negative-caching-timeout.replay.yaml
index 1c1e01e..3d90a95 100644
--- a/tests/gold_tests/cache/replay/negative-caching-timeout.replay.yaml
+++ b/tests/gold_tests/cache/replay/negative-caching-timeout.replay.yaml
@@ -76,7 +76,7 @@ sessions:
 
       # Add a delay so ATS has time to finish any caching IO for the previous
       # transaction.
-      delay: 50ms
+      delay: 100ms
 
     # 403 responses should be cached when negative caching is enabled, so this
     # should not get to the server.  But if it does, return a 404 so the test