envoyproxy/envoy

config: allow double registration with different name and deprecation

Open

#8540 aperta il 9 ott 2019

Vedi su GitHub
 (1 commento) (2 reazioni) (1 assegnatario)C++ (5373 fork)batch import
api/v3enhancementhelp wanted

Metriche repository

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

Descrizione

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

Guida contributor

Direzione di ricerca
Rivedere la discussione nella PR #8508 all'indirizzo https://github.com/envoyproxy/envoy/pull/8508#discussion r332119188 per comprendere il contesto della doppia registrazione. Le attività rimanenti sono aggiungere una statistica deprecata e implementare DEPRECATED FEATURE TEST. Esaminare i modelli di deprecazione esistenti nella codebase, come in source/common/config/, per capire come vengono utilizzate le statistiche e le macro di test per le funzionalità deprecate. Cercare file come 'source/server/config validation/api validation.h' o simili per orientamento.
Tech stack
cpp
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
55