open-telemetry/opentelemetry-php

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

Fermée

#1 314 ouverte le 20 mai 2024

Voir sur GitHubGuide contributeur
 (18 commentaires) (0 réaction) (2 personnes assignées)PHP (224 forks)auto 404
buggood first issuehelp wanted

Métriques du dépôt

Stars
 (894 étoiles)
Métriques de merge PR
 (Merge moyen 5j 5h) (5 PRs mergées en 30 j)

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?

Guide contributeur

Direction de recherche
Examinez le middleware Messenger d'auto instrumentation Symfony et ajustez le type de span en PRODUCER lors de l'envoi de messages et en CONSUMER lors du traitement, conformément à la sémantique de messagerie OpenTelemetry.
Stack technique
php
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PHP
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50