tapir icon indicating copy to clipboard operation
tapir copied to clipboard

Add an OpenTelemetry direct-style tracing interceptor

Open adamw opened this issue 1 year ago • 2 comments

Introduce a new module, tracing/opentelemetry-tracing-sync, which should include integration with OT's tracing: reading the context from the request and propagating it downstream.

The approach should target "direct-style" (that is, the Identity "effect type"), and be compatible with Loom/Ox, that is with virtual threads & scoped values. Note that the the OT Java libraries often rely on ThreadLocals, which are not Loom-compatible.

The solution should work in conjunction with https://github.com/softwaremill/sttp/issues/2314

adamw avatar Oct 16 '24 10:10 adamw

/bounty $750

adamw avatar Oct 16 '24 10:10 adamw

As we are getting first PRs for the bounties, I've published our "How to prepare a good PR" guide. I should have probably done this right away, sorry! :)

adamw avatar Oct 21 '24 10:10 adamw

/attempt #4103

Algora profile Completed bounties Tech Active attempts Options
@varshith257 17 bounties from 9 projects
Scala, Rust,
TypeScript & more
Cancel attempt

varshith257 avatar Oct 25 '24 14:10 varshith257

@adamw is this module is still needed.would love to work on this if anyone is not working .

kernel-loophole avatar Nov 16 '24 08:11 kernel-loophole

@kernel-loophole I had progress on it. Just I have open PR of other issue and I am waiting to merge it and get into this completed

varshith257 avatar Nov 16 '24 08:11 varshith257

@varshith257 hm which PR you refer to? maybe I've missed something:)

adamw avatar Nov 16 '24 14:11 adamw

@adamw https://github.com/softwaremill/tapir/pull/4122

I just made initial changes long before for this issue which I think we primarily need to work on in sttp(https://github.com/softwaremill/sttp/issues/2314 ) and need to test and evaluate my approach. Just thought to complete them (#4057 and https://github.com/softwaremill/sttp/issues/2292one by one rather than solving this set of issues (refactor of OT and this tracing/opentelemetry-tracing-syn issues simultaneously

varshith257 avatar Nov 16 '24 15:11 varshith257

/attempt #4103

Options

hoklims avatar Nov 22 '24 07:11 hoklims

[!NOTE] The user @varshith257 is already attempting to complete issue #4103 and claim the bounty. We recommend checking in on @varshith257's progress, and potentially collaborating, before starting a new solution.

algora-pbc[bot] avatar Nov 22 '24 07:11 algora-pbc[bot]

@hoklims: Reminder that in 7 days the bounty will become up for grabs, so please submit a pull request before then 🙏

algora-pbc[bot] avatar Nov 29 '24 07:11 algora-pbc[bot]

/attempt #4103

Options

promisingcoder avatar Dec 12 '24 02:12 promisingcoder

[!NOTE] The user @varshith257 is already attempting to complete issue #4103 and claim the bounty. We recommend checking in on @varshith257's progress, and potentially collaborating, before starting a new solution.

algora-pbc[bot] avatar Dec 12 '24 02:12 algora-pbc[bot]

@promisingcoder: Reminder that in 7 days the bounty will become up for grabs, so please submit a pull request before then 🙏

algora-pbc[bot] avatar Dec 19 '24 02:12 algora-pbc[bot]