From 9c3d5a76d02b5bc871bfe7a9e6acec7a89efcdd6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E7=BD=97=E6=B3=BD=E8=BD=A9?= Date: Wed, 28 Jun 2023 14:40:17 +0800 Subject: [PATCH] Update gep-1742.md to add more Nginx's timeout settings according to the suggestion in https://github.com/kubernetes-sigs/gateway-api/pull/2117#pullrequestreview-1484544095 --- geps/gep-1742.md | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/geps/gep-1742.md b/geps/gep-1742.md index 9cc1f293e9..7299f68ba3 100644 --- a/geps/gep-1742.md +++ b/geps/gep-1742.md @@ -166,7 +166,10 @@ sequenceDiagram C->>P: Finishes Headers Note right of P: client_headers_timeout deactivate P + activate P C->>P: Finishes request + deactivate P + Note right of P: client_body_timeout activate U note left of U: proxy_connect_timeout
grpc_connect_timeout deactivate U @@ -184,11 +187,14 @@ sequenceDiagram Note right of U: (between read operations)
proxy_read_timeout
grpc_read_timeout U->>P: Finishes Response deactivate U + activate P P->>C: Starts Response P->>C: Finishes Headers P->>C: Finishes Response - Note right of P: Repeat if connection sharing - Note Right of U: keepalive_timeout (if keepalive enabled) + deactivate P + Note left of P: send_timeout (only between two successive write operations) + Note right of P: Repeat if connection is shared until server's keepalive_timeout is hit + Note Right of U: upstream's keepalive_timeout (if keepalive enabled) U->>C: Connection ended deactivate U ```