envoyproxy/envoy

Cluster healthcheck uses wrong interval due to metric filtering

Open

#8771 aperta il 25 ott 2019

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
bughelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Il problema è che il controllo di integrità del cluster utilizza la metrica 'upstream cx total' per determinare l'intervallo del traffico, ma quando tale metrica viene filtrata tramite StatsMatcher, ricade erroneamente sull'intervallo predefinito 'no traffic interval'. La correzione comporta la modifica della logica di controllo di integrità in 'source/common/upstream/health checker base impl.cc' per non fare affidamento su tale metrica per il calcolo dell'intervallo, o per garantire che la metrica sia sempre disponibile. Esamina i problemi simili #8473 e #8630 per i modelli precedenti.
Tech stack
cpp
Dominio
backendinfrastructure
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Envoy configuration
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30