envoyproxy/envoy

Avoid lossy LRS during cluster updates

Open

#12 823 ouverte le 26 août 2020

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

From https://github.com/grpc/proposal/pull/200, when a cluster update for X is received and the new cluster is active after warming, the old X cluster may still be draining for an indefinite period if it has active streams. Only the stats for new requests/streams on X will participate in LRS reporting at this point. This seems to be potentially misleading when reporting load only for the new X.

There are two scenarios to consider:

  1. When the EDS service name changes. I think in this case we should generate two LRS reports, one for old cluster X (including the older EDS service name) and another for new cluster X (including the new EDS service name).
  2. When the EDS service name is the same. I think a unified LRS report should be created aggregating old/new cluster X.

Guide contributeur

Direction de recherche
Examinez l'implémentation du LRS (Load Reporting Service) dans Envoy, en particulier comment les mises à jour de cluster affectent le reporting de charge. Concentrez vous sur la logique qui gère le drainage du cluster et l'activation du nouveau cluster. Regardez le code responsable de la génération des rapports LRS lors des mises à jour de cluster et comprenez comment les changements de nom de service EDS sont traités. Tracez le chemin du code de la mise à jour CDS à la génération du rapport LRS.
Stack technique
cpp
Domaine
networking
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-2 jours
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++Envoy
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
25