envoyproxy/envoy

Cluster healthcheck uses wrong interval due to metric filtering

Open

#8.771 geöffnet am 25. Okt. 2019

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
bughelp wanted

Repository-Metriken

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

Beschreibung

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

Contributor Guide

Research-Richtung
Das Problem besteht darin, dass der Cluster Health Checker die Metrik 'upstream cx total' verwendet, um das Verkehrsintervall zu bestimmen. Wenn diese Metrik jedoch über StatsMatcher herausgefiltert wird, fällt sie fälschlicherweise auf das Standardintervall 'no traffic interval' zurück. Die Lösung besteht darin, die Health Check Logik in 'source/common/upstream/health checker base impl.cc' so zu ändern, dass sie sich nicht auf diese Metrik für die Intervallberechnung verlässt oder sicherstellt, dass die Metrik immer verfügbar ist. Überprüfen Sie ähnliche Probleme #8473 und #8630 für frühere Muster.
Tech Stack
cpp
Domain
backendinfrastructure
Issue Type
Fehler
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++Envoy configuration
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30