envoyproxy/envoy

redis outlier detection when redis is down but Envoy is up

Open

#24.215 aberto em 26 de nov. de 2022

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

In the current implementation , when server Envoy is up but redis down, we send proper error response with "UpstreamFailure" - as a result of the this Client Envoy does not detect this as outlier and eject even when outlier detection is enabled on client side.

So in cases where redis container is down but Envoy is up - Should n't this failure treated as outlier from client side (similar to 503 in http)?

Or are there any reasons why this has been implemented like this?

Guia do colaborador

Direção de pesquisa
Examine o código de detecção de outliers no filtro Redis do Envoy para entender por que as falhas upstream não são tratadas como outliers e proponha uma correção para tratar a falha do Redis como um evento de outlier.
Pilha de tecnologia
None
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.
2
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++GitEnvoy
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60