Timeouts - add test case for disabling a timeout (#2823)

* Timeouts - add test cases for disabling a timeout * reduce timeout to 1s * make all delays in timeout test 1s
kubernetes-sigs · Apr 5, 2024 · 8be7987 · 8be7987
1 parent 1646f69
commit 8be7987
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 0 deletions.
diff --git a/conformance/tests/httproute-timeout-backend-request.go b/conformance/tests/httproute-timeout-backend-request.go
@@ -59,6 +59,10 @@ var HTTPRouteTimeoutBackendRequest = suite.ConformanceTest{
 				Request:   http.Request{Path: "/backend-timeout?delay=1s"},
 				Response:  http.Response{StatusCode: 504},
 				Namespace: ns,
+			}, {
+				Request:   http.Request{Path: "/disable-backend-timeout?delay=1s"},
+				Response:  http.Response{StatusCode: 200},
+				Namespace: ns,
 			},
 		}
 

diff --git a/conformance/tests/httproute-timeout-backend-request.yaml b/conformance/tests/httproute-timeout-backend-request.yaml
@@ -16,3 +16,12 @@ spec:
       port: 8080
     timeouts:
       backendRequest: 500ms
+  - matches:
+    - path:
+        type: PathPrefix
+        value: /disabled-backend-timeout
+    backendRefs:
+    - name: infra-backend-v1
+      port: 8080
+    timeouts:
+      backendRequest: "0s"
diff --git a/conformance/tests/httproute-timeout-request.go b/conformance/tests/httproute-timeout-request.go
@@ -59,6 +59,10 @@ var HTTPRouteTimeoutRequest = suite.ConformanceTest{
 				Request:   http.Request{Path: "/request-timeout?delay=1s"},
 				Response:  http.Response{StatusCode: 504},
 				Namespace: ns,
+			}, {
+				Request:   http.Request{Path: "/disable-request-timeout?delay=1s"},
+				Response:  http.Response{StatusCode: 200},
+				Namespace: ns,
 			},
 		}
 

diff --git a/conformance/tests/httproute-timeout-request.yaml b/conformance/tests/httproute-timeout-request.yaml
@@ -16,3 +16,12 @@ spec:
       port: 8080
     timeouts:
       request: 500ms
+  - matches:
+    - path:
+        type: PathPrefix
+        value: /disable-request-timeout
+    backendRefs:
+    - name: infra-backend-v1
+      port: 8080
+    timeouts:
+      request: "0s"