envoyproxy/envoy

Cluster healthcheck uses wrong interval due to metric filtering

Open

#8 771 ouverte le 25 oct. 2019

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

Given an Envoy v1.11.1 node with cluster metrics excluded via StatsMatcher and cluster health checking enabled, when sending traffic to the cluster healthcheck is always run with no_traffic_interval 60s default interval.

The cause is the same as with https://github.com/envoyproxy/envoy/issues/8473 and https://github.com/envoyproxy/envoy/issues/8630, metrics are being used for application functionality.

In this case upstream_cx_total is the metric on which the dependency exists. Therefore the workaround is to whitelist it:

stats_config:
 stats_matcher:
  inclusion_list:
    patterns:
    - suffix: upstream_cx_total

Guide contributeur

Direction de recherche
Le problème est que le vérificateur d'intégrité du cluster utilise la métrique 'upstream cx total' pour déterminer l'intervalle de trafic, mais lorsque cette métrique est filtrée via StatsMatcher, elle revient incorrectement à l'intervalle par défaut 'no traffic interval'. La correction consiste à modifier la logique de vérification d'intégrité dans 'source/common/upstream/health checker base impl.cc' pour ne pas dépendre de cette métrique pour le calcul de l'intervalle, ou pour garantir que la métrique soit toujours disponible. Examinez les problèmes similaires #8473 et #8630 pour les modèles antérieurs.
Stack technique
cpp
Domaine
backendinfrastructure
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.
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 configuration
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30