envoyproxy/envoy

Are the lb_policy and lb_config forced to match?

Aperta

#25.583 aperta il 15 feb 2023

Vedi su GitHubGuida contributor
 (9 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/configurationarea/load balancinghelp wantedquestion

Metriche repository

Star
 (27.997 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Description: Currently in clusterInfoImpl when the lb_configs are loaded, it doesn't check if the lb_policy matches the config that's loaded. We have a separate member for the lb_policy and other members for each config type. Is there anything to stop the user from setting a round_robin config but also setting the lb_policy to least request? If there isn't, should anything be done about that?

Relevant Links: Current implementation of how the lb config members are initialized

Current location of where lb_policy is set

Guida contributor

Direzione di ricerca
Ispeziona l'implementazione corrente in source/common/upstream/upstream impl.cc per vedere se esiste una validazione che assicuri che lb policy corrisponda a lb config fornito. In caso contrario, considera l'aggiunta di un controllo nell'inizializzazione del cluster per avvisare o segnalare un errore in caso di mancata corrispondenza. Guarda come vengono gestiti altri casi simili di mancata corrispondenza di configurazione nel codebase.
Tech stack
cpp
Dominio
backend
Tipo issue
Ricerca
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++understanding of Envoy configuration
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70