envoyproxy/envoy

requests failed "above" EDS level should be reflected in load reporting

Open

#13 206 ouverte le 21 sept. 2020

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

There are at least a couple of cases where Envoy will fail requests "above" the EDS layer:

  • When the EDS resource does not exist.
  • When the cluster fails requests due to maintenance mode.

In these cases, the failed requests don't show up in LRS load reporting at all. I think this should be changed to report these as failed requests in an UpstreamLocalityStats message that does not have the locality field populated. This representation seems compatible with how we report stats for non-EDS clusters (e.g., LOGICAL_DNS clusters).

CC @htuch

Guide contributeur

Direction de recherche
Enquetez sur les deux cas (ressource EDS inexistante, mode maintenance) et modifiez le code de rapport de charge pour émettre un UpstreamLocalityStats avec une localité vide pour les requêtes échouées au niveau EDS.
Stack technique
Aucun
Domaine
backendnetworking
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.
Une demi journée
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++Envoy development setup
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40