envoyproxy / envoyproxy/envoy

SkyWalking tracer extension header support

Open
#14,933 0 comments 1 reaction 0 assignees View on GitHub
area/tracing enhancement help wanted
Dominant language
C++
Stars
28.9k
Forks
5.6k
Avg merge
1d 20h
Merged PRs (30d)
428

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

Contributor guide

Open the contributing guide

Research direction

Start in source/extensions/tracers/skywalking/skywalking_tracer_impl.cc at the linked propagation code, then compare cpp2sky's source/propagation_impl.h and the SkyWalking extension-header protocol. The work is done when the Envoy SkyWalking tracer accepts the propagated sw8-x header context and uses it consistently with the protocol.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
observability-sre
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.