open-telemetry/opentelemetry-php

[Symfony auto-instrumentation] Messenger instrumentation should use SpanKind::KIND_PRODUCER, SpanKind::KIND_CONSUMER

Fechada

#1.314 aberto em 20 de mai. de 2024

Ver no GitHubGuia do colaborador
 (18 comentários) (0 reação) (2 responsáveis)PHP (224 forks)auto 404
buggood first issuehelp wanted

Métricas do repositório

Stars
 (894 estrelas)
Métricas de merge de PR
 (Mesclagem média 5d 5h) (5 fundiu PRs em 30d)

Description

Steps to reproduce

Enable Symfony auto-instrumentation with default setup.

What is the expected behavior?

Spans when creating the message should be marked with SpanKind::KIND_PRODUCER.

Spans when handling the message should be marked with SpanKind::KIND_CONSUMER.

https://opentelemetry.io/docs/specs/semconv/messaging/messaging-spans/#span-kind

What is the actual behavior?

Everything is marked KIND_INTERNAL and it seems the consumer is currently not instrumented at all (I might be wrong here, please feel free to correct me).

Additionally, it seems to me the names of the spans are also not correct, as shown in the table in the spec linked above?

Guia do colaborador

Direção de pesquisa
Examine o middleware Messenger da auto instrumentação Symfony e ajuste o tipo de span para PRODUCER ao enviar mensagens e CONSUMER ao processar mensagens, seguindo a semântica de mensagens OpenTelemetry.
Pilha de tecnologia
php
Domain
backend
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
PHP
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50