envoyproxy/envoy

config: allow double registration with different name and deprecation

Open

#8.540 aberto em 9 de out. de 2019

Ver no GitHub
 (1 comment) (2 reactions) (1 assignee)C++ (5.373 forks)batch import
api/v3enhancementhelp wanted

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

Description:

To support extension rename with better deprecation. e.g. allow TLS transport socket to be registered as envoy.transport_socket.tls and tls and the latter should be deprecated.

https://github.com/envoyproxy/envoy/pull/8508#discussion_r332119188

Work Items:

  • double registration
  • warn log
  • deprecated stat
  • DEPRECATED_FEATURE_TEST

Guia do colaborador

Direção de pesquisa
Reveja a discussão na PR #8508 em https://github.com/envoyproxy/envoy/pull/8508#discussion r332119188 para entender o contexto do registro duplo. Os itens de trabalho restantes são adicionar uma estatística obsoleta e implementar DEPRECATED FEATURE TEST. Examine os padrões de depreciação existentes na base de código, como em source/common/config/, para entender como as estatísticas e macros de teste são usadas para recursos obsoletos. Procure arquivos como 'source/server/config validation/api validation.h' ou similares para orientação.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
55