Skip to content
This repository has been archived by the owner on Nov 30, 2021. It is now read-only.

feat(*) support OT headers #103

Merged
merged 3 commits into from
Feb 17, 2021
Merged

feat(*) support OT headers #103

merged 3 commits into from
Feb 17, 2021

Conversation

kikito
Copy link
Member

@kikito kikito commented Feb 16, 2021

This is a retake on #95. I have:

  • Rebased it over master
  • Added interaction tests between ot and jaeger headers (they were not present when the original pr was submitted)
  • Addressed some of @Tieske 's comments

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@kikito kikito force-pushed the lightstep-support-ot-headers branch from 12ad7ba to 2c39745 Compare February 16, 2021 09:41
@kikito kikito force-pushed the lightstep-support-ot-headers branch from 2c39745 to 178ad65 Compare February 16, 2021 09:51
@kikito kikito marked this pull request as ready for review February 16, 2021 16:50
@kikito kikito changed the title Support OT headers feat(*) support OT headers Feb 17, 2021
@kikito kikito merged commit 39d5cbc into master Feb 17, 2021
@kikito kikito deleted the lightstep-support-ot-headers branch February 17, 2021 11:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants