envoyproxy/envoy

conditional retry condition in retry policy

Open

#9.946 geöffnet am 6. Feb. 2020

Auf GitHub ansehen
 (3 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/retrydesign proposalhelp wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

Description:

https://github.com/envoyproxy/envoy/blob/abb1f969fe131c7a47da5c1ad1577be142e1ea8a/api/envoy/config/route/v3/route_components.proto#L884

defines retry policy for HTTP request. It doesn't support configure different retry policies for different HTTP methods. Users might want to retry on connection failure for all methods but 5xx for idempotent methods only. I think the retry policy should provide such function to allow configure different retry policies for different HTTP methods. More generally, the retry policy should be able to support conditional retry condition.

Relevant Links: https://github.com/istio/istio/issues/13851

Contributor Guide