envoyproxy/envoy

Enforce prevention of multiple instances of a locality in the same priority

Open

#20 768 ouverte le 11 avr. 2022

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
apihelp 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

Title: Enforce prevention of multiple instances of a locality in the same priority

Description: PR #20574 refined the LocalityLbEndpoints structure to be defined as:

// A group of endpoints belonging to a Locality.
// One can have multiple LocalityLbEndpoints for a locality, but only if
// they have different priorities.

but Envoy allows to have multiple LocalityLbEndpoints for a locality if the different groups have different weights or different priorities.

Envoy should be more restrictive, and confirm to the revised comment.

cc @markdroth

Guide contributeur

Direction de recherche
Examinez la logique de validation actuelle de LocalityLbEndpoints dans Envoy et ajoutez une verification pour empecher plusieurs instances de la meme localite avec la meme priorite. Consultez la PR #20574 pour le contexte.
Stack technique
cpp
Domaine
backendnetworking
Type d'issue
Bug
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-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60