From 514c56668aa718581a0b3163a18a3b3e5d1cfa7e Mon Sep 17 00:00:00 2001 From: Dhi Aurrahman Date: Fri, 12 Jan 2018 09:22:14 +0700 Subject: [PATCH 1/3] Add gateway error retry-on policy Signed-off-by: Dhi Aurrahman --- docs/root/configuration/http_filters/router_filter.rst | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/docs/root/configuration/http_filters/router_filter.rst b/docs/root/configuration/http_filters/router_filter.rst index 73d3e528b..1fa43de77 100644 --- a/docs/root/configuration/http_filters/router_filter.rst +++ b/docs/root/configuration/http_filters/router_filter.rst @@ -80,6 +80,10 @@ using a ',' delimited list. The supported policies are: :ref:`config_http_filters_router_x-envoy-upstream-rq-timeout-ms` is an outer time limit for a request, including any retries that take place. +gateway-error + Envoy will attempt a retry if the upstream server responds with 502, 503 or 504 response code. This policy should have + the same behavior with 5xx, despite the smaller number of response codes to be checked. + connect-failure Envoy will attempt a retry if a request is failed because of a connection failure to the upstream server (connect timeout, etc.). (Included in *5xx*) From 2000b00db187c5a4aa8604f1a9e63605d10c3355 Mon Sep 17 00:00:00 2001 From: Dhi Aurrahman Date: Sat, 13 Jan 2018 05:48:24 +0700 Subject: [PATCH 2/3] Rephrase the gateway-error retry-on policy. Signed-off-by: Dhi Aurrahman --- docs/root/configuration/http_filters/router_filter.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/root/configuration/http_filters/router_filter.rst b/docs/root/configuration/http_filters/router_filter.rst index 1fa43de77..b751925a9 100644 --- a/docs/root/configuration/http_filters/router_filter.rst +++ b/docs/root/configuration/http_filters/router_filter.rst @@ -81,8 +81,8 @@ using a ',' delimited list. The supported policies are: request, including any retries that take place. gateway-error - Envoy will attempt a retry if the upstream server responds with 502, 503 or 504 response code. This policy should have - the same behavior with 5xx, despite the smaller number of response codes to be checked. + Envoy will attempt a retry if the upstream server responds with 502, 503, or 504 response code. + This policy is similar to the *5xx* policy but will only retry requests that result in a 502, 503, or 504. connect-failure Envoy will attempt a retry if a request is failed because of a connection failure to the upstream From 6deceeab2d6ca5f2d504a7c506674fcd52e0ea6f Mon Sep 17 00:00:00 2001 From: Dhi Aurrahman Date: Sun, 14 Jan 2018 02:52:51 +0700 Subject: [PATCH 3/3] Simplify the gateway-error policy description Signed-off-by: Dhi Aurrahman --- docs/root/configuration/http_filters/router_filter.rst | 1 - 1 file changed, 1 deletion(-) diff --git a/docs/root/configuration/http_filters/router_filter.rst b/docs/root/configuration/http_filters/router_filter.rst index b751925a9..b02a4fd1b 100644 --- a/docs/root/configuration/http_filters/router_filter.rst +++ b/docs/root/configuration/http_filters/router_filter.rst @@ -81,7 +81,6 @@ using a ',' delimited list. The supported policies are: request, including any retries that take place. gateway-error - Envoy will attempt a retry if the upstream server responds with 502, 503, or 504 response code. This policy is similar to the *5xx* policy but will only retry requests that result in a 502, 503, or 504. connect-failure