envoyproxy/envoy

Avoid lossy LRS during cluster updates

Open

#12.823 aberto em 26 de ago. de 2020

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

From https://github.com/grpc/proposal/pull/200, when a cluster update for X is received and the new cluster is active after warming, the old X cluster may still be draining for an indefinite period if it has active streams. Only the stats for new requests/streams on X will participate in LRS reporting at this point. This seems to be potentially misleading when reporting load only for the new X.

There are two scenarios to consider:

  1. When the EDS service name changes. I think in this case we should generate two LRS reports, one for old cluster X (including the older EDS service name) and another for new cluster X (including the new EDS service name).
  2. When the EDS service name is the same. I think a unified LRS report should be created aggregating old/new cluster X.

Guia do colaborador

Direção de pesquisa
Examine a implementação do LRS (Serviço de Relatório de Carga) no Envoy, especificamente como as atualizações de cluster afetam o relatório de carga. Concentre se na lógica que lida com o dreno do cluster e a ativação do novo cluster. Veja o código responsável por gerar relatórios LRS durante as atualizações de cluster e entenda como as mudanças no nome do serviço EDS são tratadas. Rastreie o caminho do código da atualização CDS até a geração do relatório LRS.
Pilha de tecnologia
cpp
Domain
networking
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.
Recente
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
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
25