envoyproxy/envoy

requests failed "above" EDS level should be reflected in load reporting

Open

#13.206 aperta il 21 set 2020

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/cluster_managerbughelp wanted

Metriche repository

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

Descrizione

There are at least a couple of cases where Envoy will fail requests "above" the EDS layer:

  • When the EDS resource does not exist.
  • When the cluster fails requests due to maintenance mode.

In these cases, the failed requests don't show up in LRS load reporting at all. I think this should be changed to report these as failed requests in an UpstreamLocalityStats message that does not have the locality field populated. This representation seems compatible with how we report stats for non-EDS clusters (e.g., LOGICAL_DNS clusters).

CC @htuch

Guida contributor

Direzione di ricerca
Investigare i due casi (risorsa EDS inesistente, modalità manutenzione) e modificare il codice di report del carico per emettere un UpstreamLocalityStats con località vuota per le richieste fallite a livello EDS.
Tech stack
Nessuno
Dominio
backendnetworking
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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++Envoy development setup
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40