envoyproxy/envoy

Custom clusters do not support endpoint discovery through EDS

Open

#15.663 geöffnet am 25. März 2021

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
enhancementhelp wanted

Repository-Metriken

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

Beschreibung

Title: Custom clusters do not support endpoint discovery through EDS

Description: Usage of eds_cluster_config is restricted to clusters that have type explicitly set to EDS. Envoy also uses this type for determining whether clusters are initialized during the primary or secondary phase, meaning it is not possible to write a custom cluster type that performs endpoint discovery via EDS.

Instead, I think the custom cluster type should specify whether it utilizes EDS for endpoint discovery, and if so, Envoy should treat it the same way as it does the native EDS cluster (i.e. the cluster is permitted to use eds_cluster_config, it is initialized during secondary phase if the config source is not file-based, etc.).

Contributor Guide

Research-Richtung
Das Issue schlägt vor, benutzerdefinierten Clustertypen die Verwendung von EDS für die Endpunktentdeckung zu ermöglichen. Zu untersuchende Schlüsselbereiche umfassen die Cluster Initialisierungsphase (primär vs. sekundär) in source/common/upstream/cluster manager impl.cc und die eds cluster config Validierung in source/common/upstream/eds cluster.cc. Der Kommentar des Maintainers schlägt vor, ein Flag hinzuzufügen, um die EDS Nutzung anzuzeigen. Untersuchen Sie, wie native EDS Cluster behandelt werden und wie dies auf benutzerdefinierte Cluster erweitert werden kann.
Tech Stack
cpp
Domain
backendinfrastructure
Issue Type
Funktion
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-2 Tage
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++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40