envoyproxy/envoy

Tracing: upstream_cluster attribute missing when backend has no healthy replicas

Aberta

#25.887 aberto em 2 de mar. de 2023

Ver no GitHubGuia do colaborador
 (4 comentários) (2 reações) (0 responsável)C++ (5.373 forks)batch import
area/routerenhancementhelp wanted

Métricas do repositório

Stars
 (27.997 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Title: Tracing: upstream_cluster attribute missing when backend has no healthy replicas

Description:

Given an envoy cluster of "foo_cluster" with some number of backends that is addressed via a k8s service name "foo-service".

If "foo_cluster" doesn't have any healthy backends (e.g. they are missing entirely, or maybe crash looping), spans generated by envoy on traffic that matches "foo_cluster" are missing the "upstream_cluster" attribute.

It would be helpful if "upstream_cluster" was always populated.

Guia do colaborador

Direção de pesquisa
Investigue o código de rastreamento do Envoy para identificar onde upstream cluster é definido. Procure condições que pulam a definição do atributo quando não há hosts upstream saudáveis disponíveis. A correção provavelmente envolve garantir que o atributo seja sempre preenchido, independentemente do status de saúde. Revise os arquivos relevantes em source/common/tracing ou source/server.
Pilha de tecnologia
cpp
Domain
backendapi
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-3 horas
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++EnvoyTracing
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50