envoyproxy/envoy

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

Open

#20.768 geöffnet am 11. Apr. 2022

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
apihelp wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die aktuelle Validierungslogik für LocalityLbEndpoints in Envoy und füge eine Prüfung hinzu, um mehrere Instanzen derselben Lokalität mit derselben Priorität zu verhindern. Siehe PR #20574 für den Kontext.
Tech Stack
cpp
Domain
backendnetworking
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60