envoyproxy/envoy

Are the lb_policy and lb_config forced to match?

Open

#25.583 geöffnet am 15. Feb. 2023

Auf GitHub ansehen
 (9 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/configurationarea/load balancinghelp wantedquestion

Repository-Metriken

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

Beschreibung

Description: Currently in clusterInfoImpl when the lb_configs are loaded, it doesn't check if the lb_policy matches the config that's loaded. We have a separate member for the lb_policy and other members for each config type. Is there anything to stop the user from setting a round_robin config but also setting the lb_policy to least request? If there isn't, should anything be done about that?

Relevant Links: Current implementation of how the lb config members are initialized

Current location of where lb_policy is set

Contributor Guide

Research-Richtung
Überprüfen Sie die aktuelle Implementierung in source/common/upstream/upstream impl.cc, ob es eine Validierung gibt, die sicherstellt, dass lb policy mit dem bereitgestellten lb config übereinstimmt. Falls nicht, erwägen Sie, eine Prüfung in der Cluster Initialisierung hinzuzufügen, um bei Nichtübereinstimmung eine Warnung oder einen Fehler zu geben. Sehen Sie sich an, wie andere ähnliche Konfigurationsinkonsistenzen in der Codebasis behandelt werden.
Tech Stack
cpp
Domain
backend
Issue Type
Recherche
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
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.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++understanding of Envoy configuration
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70