envoyproxy/envoy

SkyWalking tracer extension header support

Open

#14 933 ouverte le 4 févr. 2021

Voir sur GitHub
 (0 commentaires) (1 réaction) (0 assignés)C++ (5 373 forks)batch import
area/tracingenhancementhelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

In current implementation, it can't accept and use context packed on sw8-x header. This feature is already supported on cpp2sky, then just to extract propagated header and use them.

https://github.com/envoyproxy/envoy/blob/main/source/extensions/tracers/skywalking/skywalking_tracer_impl.cc#L53 https://github.com/SkyAPM/cpp2sky/blob/main/source/propagation_impl.h#L55-L63 https://github.com/apache/skywalking/blob/08781b41a8255bcceebb3287364c81745a04bec6/docs/en/protocols/Skywalking-Cross-Process-Propagation-Headers-Protocol-v3.md#extension-header-item

Guide contributeur

Direction de recherche
Le ticket demande l'ajout de la prise en charge de l'en tête `sw8 x` dans le traceur SkyWalking d'Envoy. Le corps comprend des liens vers l'implémentation C++ pertinente dans cpp2sky et la source du traceur Envoy, fournissant des exemples clairs. La recherche doit se concentrer sur la compréhension de l'implémentation actuelle du traçage d'Envoy et sur l'adaptation de la logique de propagation des en têtes depuis cpp2sky. Le ticket est inactif sans activité récente, donc la mise en œuvre nécessiterait une validation indépendante.
Stack technique
cpp
Domaine
observability
Type d'issue
Fonctionnalité
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.
Une demi journée
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
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65