OpenFeign/feign

Feign observation span is not put in "scope" - loggers do not see current spanId

Open

#2 406 ouverte le 5 mai 2024

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Java (9 070 stars) (1 911 forks)batch import
bughelp wanted

Description

Despite new observation is started around feign call, MDC still contains "parent" spanId.

Code to reproduce:

https://github.com/norbson/feign-tracing-issue

run and call http://localhost:8080/test

Result: Screenshot from 2024-05-05 18-04-03

Guide contributeur

Stack technique
javaspring
Domaine
backendapiobservability
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
JavaSpringMicrometer Observation
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50
Direction de recherche
The issue is that the Feign observation span is not placed in scope, causing MDC to retain the parent spanId. Investigate the Feign observation instrumentation, likely in the `feign micrometer` module. Check if the observation is being opened and closed correctly, and ensure that the span is set as the current span in the tracing context. Look at the Micrometer Observation API's `Observation.Scope` usage and compare with similar tracing integrations. The reproduction repository is at https://github.com/norbson/feign tracing issue.