envoyproxy/envoy

Envoy and control-plane xDS over ADS protocol deadlock when filtering stats

Open

#9.856 aberto em 28 de jan. de 2020

Ver no GitHub
 (12 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/statsarea/xdsbughelp 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

Description: When using a stats_matcher inclusion_list, Envoy using the ADS variant of the xDS protocol will fail to receive cluster updates from the control-plane. After receiving an initial Cluster list, Envoy will no longer receive cluster updates. When paired with envoyproxy/go-control-plane in ADS mode, this can cause EDS updates to stall if the cluster list is later expanded.

Repro steps: A docker-compose reproduction of the issue with Envoy 1.12.2 and 1.13.0

Relevant Links:

Guia do colaborador

Direção de pesquisa
Investigue o deadlock entre o matcher de estatísticas do Envoy e as atualizações ADS xDS. Revise a configuração de reprodução em https://github.com/dfjones/envoy stats filter repro. Examine o código fonte do Envoy em src/envoy/config/ e src/common/upstream/ em busca de potenciais condições de deadlock. Consulte a issue #119 do go control plane para o comportamento de consistência. Procure por problemas de ordem de bloqueio quando a filtragem de estatísticas dispara um callback que afeta o processamento da resposta xDS.
Pilha de tecnologia
cppgo
Domain
infrastructurebackend
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.
4
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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.
Principalmente claro
Prerequisites
C++GoDocker
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40