sttp icon indicating copy to clipboard operation
sttp copied to clipboard

Add an Ox/direct-style-compatible OpenTelemetry tracing backend

Open adamw opened this issue 1 year ago • 3 comments

Currently sttp-client only ships with OT+ZIO tracing integration.

The goal of the issue is to create a new module, opentelemetry-tracing-sync-backend, which will provide integration with blocking/direct-style OpenTelemetry tracing, as well as reference documentation showcasing how to use the module.

Moreover, a first step is to suggest a way of propagating the context (setting appropriate headers, including identifiers/correlation ids) that is compatible with virtual threads & scoped values, as used by Loom and Ox. Note that the the OT Java libraries often rely on ThreadLocals, which are not Loom-compatible.

See also https://github.com/softwaremill/tapir/issues/4103

adamw avatar Oct 16 '24 09:10 adamw

/bounty $750

adamw avatar Oct 16 '24 10:10 adamw

Once #2292 is done I will jump here to tackle this.

varshith257 avatar Oct 16 '24 13:10 varshith257

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

Hey @adamw is this still available ?

stdthoth avatar Nov 21 '24 11:11 stdthoth

@stdthoth it's in WIP. Will raise PR soon I think I have to push a PR so that others know I am working on it but just a few things need a fix and will draft PR

varshith257 avatar Nov 21 '24 11:11 varshith257

@stdthoth as long as a PR isn't merged it is available - I've setup attempts (with /attempt) to time out after two weeks, that could provide some indication. @varshith257 I don't think you've attempted this issue?

Anyway, the first (good!) PR to get merged wins :)

adamw avatar Nov 21 '24 13:11 adamw

I haven't attempted due to algora bot limits the attempts and I haven't done this to make sure not make any messy with algora bot 😀

Anyway /attempt #2314

Algora profile Completed bounties Tech Active attempts Options
@varshith257    1 softwaremill bounty
+ 19 bounties from 9 projects
Rust, Scala,
Go & more
Cancel attempt

varshith257 avatar Nov 21 '24 13:11 varshith257

@varshith257: 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 28 '24 13:11 algora-pbc[bot]

The bounty is up for grabs! Everyone is welcome to /attempt #2314 🙌

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

/attempt #2314

Algora profile Completed bounties Tech Active attempts Options
@stdthoth 1 bounty from 1 project
Cancel attempt

stdthoth avatar Dec 06 '24 11:12 stdthoth

@stdthoth: 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 13 '24 11:12 algora-pbc[bot]

The bounty is up for grabs! Everyone is welcome to /attempt #2314 🙌

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