Skip to content

Propagate Trace context in exit spans#1358

Merged
russcam merged 3 commits intoelastic:masterfrom
russcam:propagate-trace-context
Aug 4, 2021
Merged

Propagate Trace context in exit spans#1358
russcam merged 3 commits intoelastic:masterfrom
russcam:propagate-trace-context

Conversation

@russcam
Copy link
Copy Markdown
Contributor

@russcam russcam commented Jul 26, 2021

This commit propagates the Trace Context for
exit span outgoing requests, without constructing
a child HTTP span, as per current behaviour.

The Trace context propagated is retrieved from
the current span.

Closes #1350
Closes #1344

This commit propagates the Trace Context for
exit span outgoing requests, without constructing
a child HTTP span, as per current behaviour.

The Trace context propagated is retrieved from
the current span.

Closes elastic#1350
@ghost
Copy link
Copy Markdown

ghost commented Jul 26, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-08-04T00:19:07.608+0000

  • Duration: 51 min 3 sec

  • Commit: ed38b13

Test stats 🧪

Test Results
Failed 0
Passed 19639
Skipped 126
Total 19765

Trends 🧪

Image of Build Times

Image of Tests

@gregkalapos gregkalapos self-requested a review July 26, 2021 11:25
Copy link
Copy Markdown
Contributor

@gregkalapos gregkalapos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@russcam
Copy link
Copy Markdown
Contributor Author

russcam commented Aug 3, 2021

jenkins run tests

@russcam russcam merged commit fd8ac01 into elastic:master Aug 4, 2021
@russcam russcam deleted the propagate-trace-context branch August 4, 2021 01:16
@russcam russcam added v1.11.1 enhancement New feature or request labels Sep 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Always propagate W3C headers to downstream services Header Propagation does not work for Elasticsearch and Azure

3 participants