envoyproxy/envoy

Custom clusters do not support endpoint discovery through EDS

Open

#15 663 ouverte le 25 mars 2021

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
enhancementhelp 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: 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.).

Guide contributeur

Direction de recherche
Cette issue propose de permettre aux types de cluster personnalisés d'utiliser EDS pour la découverte de points de terminaison. Les domaines clés à étudier incluent la phase d'initialisation du cluster (primaire vs secondaire) dans source/common/upstream/cluster manager impl.cc et la validation de eds cluster config dans source/common/upstream/eds cluster.cc. Le commentaire du mainteneur suggère d'ajouter un indicateur pour signaler l'utilisation d'EDS. Explorez comment les clusters EDS natifs sont gérés et comment étendre cela aux clusters personnalisés.
Stack technique
cpp
Domaine
backendinfrastructure
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
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.
Claire
Prérequis
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40