envoyproxy/envoy

tracing: Set span.kind tag for opencensus tracer when creating client/server spans

Open

#12.538 aberto em 7 de ago. de 2020

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

Currently OpenTracing tracers and the zipkin tracer derives the span.kind (or cs/cr/ss/sr annotations) from the tracing config "operation_name" - however opencensus tracer doesn't have this tag set.

The proxy needs to explicitly set this tag when creating the span. This is similiar to #3435 which was resolved in 42a9cae38dd3bf5264fb83f64e310b0d76ea4257 (but this time just for opencensus)

Guia do colaborador

Direção de pesquisa
Estude a correção para OpenTracing no commit 42a9cae38dd3bf5264fb83f64e310b0d76ea4257 e aplique uma abordagem semelhante ao tracer OpenCensus. Localize onde os spans são criados no tracer OpenCensus e adicione lógica para definir span.kind com base no nome da operação (cliente/servidor).
Pilha de tecnologia
cpp
Domain
networkingbackend
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.
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++Envoy conceptsTracing concepts
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60