envoyproxy/envoy

Migrating `RateLimit` to typed filter config phase 2: Envoy consumes the new field

Open

#18.118 aperta il 14 set 2021

Vedi su GitHub
 (2 commenti) (0 reazioni) (1 assegnatario)C++ (5373 fork)batch import
area/ratelimithelp wantedtech debt

Metriche repository

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

Descrizione

Title: Migrating RateLimit to typed filter config phase 2: Envoy consumes the new field

Description: This is to track the second phase of migrating Ratelimit to RateLimitPerRoute. In the first phase, the field is added to typed filter config. Now this work needs to be completed as described here: https://github.com/envoyproxy/envoy/pull/18044#discussion_r706285153.

Relevant Links:

  • #14732

cc @yanavlasov @tyxia @mattklein123 @markdroth

Guida contributor

Direzione di ricerca
Esamina la discussione nella PR #18044 (https://github.com/envoyproxy/envoy/pull/18044), in particolare il commento alla riga 706285153. Comprendi l'attuale campo RateLimitPerRoute aggiunto nella fase 1 e implementa il consumo della nuova configurazione di filtro tipizzata. Rivedi l'issue di riferimento #14732 per il contesto. Il maintainer assegnato potrebbe fornire indicazioni; fai domande chiarificatrici sul comportamento previsto.
Tech stack
c
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 filter configProtobuf
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30