envoyproxy/envoy

enhance router_check_tool to allow going through listeners as well

Open

#12.393 aperta il 30 lug 2020

Vedi su GitHub
 (7 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/route_check_toolhelp wanted

Metriche repository

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

Descrizione

Currently, router_check_tool takes in a route as input. This makes the tool somewhat whitebox - the user first needs to figure out which route they care about, then they can write a test for it. Ideally, its completely black box - the user gives the full Envoy config (really just Listeners and Routes) and a set of queries, and checks the result.

This is useful as well to test complex listeners with many different filter chain matches.

Alternatives considered:

  • Implement the logic for determining a route from the request ourselves. This would involve re-implementing the filter chain/listener matching logic which is likely to be incorrect and fall out of sync
  • Use the tool as is. This doesn't mean the requirement of black box testing, relies test writers to know Envoy implementation details
  • Send real traffic against a real envoy instead. This is slower and more complex

Guida contributor

Direzione di ricerca
Studia il codice esistente di router check tool e comprendi come attualmente accetta una route. Poi modificalo per accettare una configurazione Envoy completa (listeners e routes) e implementa la logica per selezionare la route appropriata dalle richieste, sfruttando la logica di corrispondenza della catena di filtri di Envoy quando possibile.
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.
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 configrouter check tool
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40