athens icon indicating copy to clipboard operation
athens copied to clipboard

Published 20 hours ago • verified publisher icon gomods

chore: use open telemetry bridge

Open uhthomas opened this issue 1 year ago • 5 comments

What is the problem I am trying to address?

As described in the linked issue, OpenCensus is to be replaced with OpenTelemetry. This can be tricky to do in a non-breaking way, and so the easiest thing to do for now is to use the OpenCensus bridge.

https://opentelemetry.io/blog/2023/sunsetting-opencensus/

https://github.com/open-telemetry/opentelemetry-go/tree/ac5639159f9c8ec161a93b1990e878c74022d49b/bridge/opencensus

How is the fix applied?

See above.

What GitHub issue(s) does this PR fix or close?

Fixes: #1771

uhthomas avatar Sep 14 '23 12:09 uhthomas

Please resolve conflicts.

manugupt1 avatar Sep 20 '23 20:09 manugupt1

@manugupt1 Apologies, this is a draft PR. I am not sure it's ready for review.

uhthomas avatar Sep 20 '23 20:09 uhthomas

Oh sorry! I missed it too :) thanks for helping in the last few days with the project.

manugupt1 avatar Sep 20 '23 20:09 manugupt1

Not a problem, thank you for taking the time to review my changes! 🙂

uhthomas avatar Sep 20 '23 20:09 uhthomas

@uhthomas If you're willing to fix this PR up and have a way to test it (OpenCensus & OTel) I'm happy to help get it into our next milestone

matt0x6F avatar Feb 19 '24 03:02 matt0x6F