envoyproxy/envoy

Cluster healthcheck uses wrong interval due to metric filtering

Open

#8.771 aberto em 25 de out. de 2019

Ver no GitHub
 (6 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
O problema é que o verificador de integridade do cluster usa a métrica 'upstream cx total' para determinar o intervalo de tráfego, mas quando essa métrica é filtrada via StatsMatcher, ela recua incorretamente para o padrão 'no traffic interval'. A correção envolve modificar a lógica de verificação de integridade em 'source/common/upstream/health checker base impl.cc' para não depender dessa métrica para o cálculo do intervalo, ou para garantir que a métrica esteja sempre disponível. Revise problemas semelhantes #8473 e #8630 para padrões anteriores.
Pilha de tecnologia
cpp
Domain
backendinfrastructure
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Envoy configuration
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30