envoyproxy/envoy

requests failed "above" EDS level should be reflected in load reporting

Open

#13.206 aberto em 21 de set. de 2020

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

There are at least a couple of cases where Envoy will fail requests "above" the EDS layer:

  • When the EDS resource does not exist.
  • When the cluster fails requests due to maintenance mode.

In these cases, the failed requests don't show up in LRS load reporting at all. I think this should be changed to report these as failed requests in an UpstreamLocalityStats message that does not have the locality field populated. This representation seems compatible with how we report stats for non-EDS clusters (e.g., LOGICAL_DNS clusters).

CC @htuch

Guia do colaborador

Direção de pesquisa
Investigue os dois casos (recurso EDS inexistente, modo de manutenção) e modifique o código de relatório de carga para emitir um UpstreamLocalityStats com localidade vazia para solicitações com falha na camada EDS.
Pilha de tecnologia
None
Domain
backendnetworking
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.
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.
Claro
Prerequisites
C++Envoy development setup
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40