envoyproxy/envoy

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

Open

#18 118 ouverte le 14 sept. 2021

Voir sur GitHub
 (2 commentaires) (0 réactions) (1 assigné)C++ (5 373 forks)batch import
area/ratelimithelp wantedtech debt

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Examinez la discussion dans la PR #18044 (https://github.com/envoyproxy/envoy/pull/18044), en particulier le commentaire à la ligne 706285153. Comprenez le champ actuel RateLimitPerRoute ajouté dans la phase 1 et implémentez la consommation de la nouvelle configuration de filtre typée. Consultez l'issue référencée #14732 pour le contexte. Le mainteneur assigné peut fournir des conseils ; posez des questions de clarification sur le comportement attendu.
Stack technique
c
Domaine
backendinfrastructure
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
C++Envoy filter configProtobuf
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30