envoyproxy/envoy

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

Open

#18.118 aberto em 14 de set. de 2021

Ver no GitHub
 (2 comments) (0 reactions) (1 assignee)C++ (5.373 forks)batch import
area/ratelimithelp wantedtech debt

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Examine a discussão no PR #18044 (https://github.com/envoyproxy/envoy/pull/18044), especialmente o comentário na linha 706285153. Entenda o campo atual RateLimitPerRoute adicionado na fase 1 e implemente o consumo da nova configuração de filtro tipada. Revise a issue referenciada #14732 para contexto. O mantenedor atribuído pode fornecer orientação; faça perguntas esclarecedoras sobre o comportamento esperado.
Pilha de tecnologia
c
Domain
backendinfrastructure
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
C++Envoy filter configProtobuf
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30