-
Notifications
You must be signed in to change notification settings - Fork 458
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
chore: add test for merging baggage entries #3256
Conversation
Datadog ReportBranch report: ✅ 0 Failed, 5500 Passed, 72 Skipped, 10m 26.77s Total Time |
BenchmarksBenchmark execution time: 2025-04-02 13:51:32 Comparing candidate commit 6809327 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 61 metrics, 1 unstable metrics. |
/merge |
View all feedbacks in Devflow UI.
This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
devflow unqueued this merge request: It did not become mergeable within the expected time |
Co-authored-by: Dario Castañé <[email protected]>
What does this PR do?
Add a test to check that if baggage is set before a request is sent, it is merged into the baggage inside the request headers as well.
Motivation
Reviewer's Checklist
v2-dev
branch and reviewed by @DataDog/apm-go.Unsure? Have a question? Request a review!