envoyproxy/envoy

conditional retry condition in retry policy

Open

#9946 aperta il 6 feb 2020

Vedi su GitHub
 (3 commenti) (2 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/retrydesign proposalhelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Investigare l'attuale implementazione della politica di retry in Envoy, in particolare la definizione protobuf in route components.proto e la logica di retry nel codice C++. Comprendere come vengono gestiti i metodi HTTP e proporre un progetto per condizioni di retry condizionali che permettono politiche diverse per metodo o condizioni personalizzate.
Tech stack
cpp
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++ProtobufEnvoy
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
25