envoyproxy/envoy

Envoy should NACK an EDS update with duplicate addresses

Open

#23.445 aperta il 11 ott 2022

Vedi su GitHub
 (2 commenti) (0 reazioni) (1 assegnatario)C++ (5373 fork)batch import
area/edsbughelp wanted

Metriche repository

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

Descrizione

Currently, if there are multiple addresses in the same EDS resource, Envoy will ignore all but the first occurrance. This behavior seems fairly arbitrary and surprising, especially because the different occurrances of the address could be in different localities or priorities.

I think in this case, Envoy should NACK the EDS update.

CC @kyessenov

Guida contributor

Direzione di ricerca
Esaminare il codice di gestione dell'aggiornamento EDS in envoy/source/common/upstream/eds.cc, in particolare la logica che ignora gli indirizzi duplicati. Tracciare il percorso dall'analisi della risposta EDS alla generazione del NACK. Comprendere il framework di validazione e come rifiutare un aggiornamento con indirizzi duplicati attivando un NACK.
Tech stack
Nessuno
Dominio
infrastructure
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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 internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50