envoyproxy/envoy

config: allow double registration with different name and deprecation

Open

#8 540 ouverte le 9 oct. 2019

Voir sur GitHub
 (1 commentaire) (2 réactions) (1 assigné)C++ (5 373 forks)batch import
api/v3enhancementhelp wanted

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

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

Guide contributeur

Direction de recherche
Examinez la discussion dans la PR #8508 à l'adresse https://github.com/envoyproxy/envoy/pull/8508#discussion r332119188 pour comprendre le contexte de l'enregistrement double. Les tâches restantes consistent à ajouter une statistique obsolète et à implémenter DEPRECATED FEATURE TEST. Étudiez les modèles d'obsolescence existants dans la base de code, comme dans source/common/config/, pour comprendre comment les statistiques et les macros de test sont utilisées pour les fonctionnalités obsolètes. Recherchez des fichiers tels que 'source/server/config validation/api validation.h' ou similaires pour vous guider.
Stack technique
cpp
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55