envoyproxy/envoy

Refactor outlier detection configuration fields

Open

#8219 aperta il 11 set 2019

Vedi su GitHub
 (5 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
api/v3help wanted

Metriche repository

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

Descrizione

Title: Refactor outlier detection config message fields

Description: As discussed in https://github.com/envoyproxy/envoy/pull/8130#discussion_r320832679, the outlier detection configuration fields are a bit messy and could use a cleanup. The main work here is to de-dup all of the options into a common message around enabled, local origin split, etc. and then hopefully just have a single field per actual ejection type. This would avoid an explosion of almost identical fields for each different ejection type.

Guida contributor

Direzione di ricerca
Studia i messaggi di configurazione del rilevamento outlier esistenti nel codice sorgente di Envoy, concentrandoti sulla discussione nella PR #8130. Comprendi la duplicazione tra i campi per diversi tipi di espulsione e proponi una struttura unificata con campi comuni come `enabled`, `local origin split`, ecc. Pianifica di creare un messaggio comune e poi avere un singolo campo per tipo di espulsione.
Tech stack
cpp
Dominio
backendinfrastructure
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C++Envoy architecture
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50