-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat: Add OTel Tracing Headers and Baggage with CueX Requests #115
Conversation
4cebeb5
to
fd8a668
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #115 +/- ##
==========================================
+ Coverage 86.46% 86.71% +0.24%
==========================================
Files 81 82 +1
Lines 3959 4071 +112
==========================================
+ Hits 3423 3530 +107
- Misses 402 408 +6
+ Partials 134 133 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
fd8a668
to
7122881
Compare
7122881
to
03df3ab
Compare
Signed-off-by: Brian Kane <[email protected]>
03df3ab
to
95cead8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding it!
Description of your changes
Adds tracing and baggage (app context) support using OTel for outbound CueX requests.
The externalserver package has been updated to reconstruct the context from the received headers.
I have:
How has this code been tested
PR includes unit tests for both the outbound and inbound functionality.
Manual testing has been conducted to ship application workflow context & trace IDs from a Kubevela Application to a CueX server.
Special notes for your reviewer
N/A