-
Notifications
You must be signed in to change notification settings - Fork 327
OpenTracing implementation on top of OpenCensus trace #502
Comments
I've written a few open tracing implementations so I'd be happy to do this. There are a couple of incompatibilities that I'd love some guidance on:
The primary behavior that will be odd for people using the OC OT implementation imho is that log messages which previously contained baggage would now just contain the log message (plus attributes). I know we put things like tenantID and userID into the baggage so losing these would be a showstopper. |
If I put my marketing hat on. If you really want OC to be widely adopted, perhaps a parallel approach would be to write native OC implementations for those other frameworks. Otherwise, there's really no reason to use OC directly if you can always use the OT adapter. |
We are worried that having two trace recording APIs in this repo will cause confusion for users. We are looking for an appropriate place to put this work. |
Fair enough. Let me know what you'd like me to do. |
@savaki we are going to create an "opencensus-contrib" organization to put all the integrations (including exporters) and I think this can go there. You can keep the PR open for now if you like or publish it to a repo in your account that we can then clone to the new org once it has been created. It should be created this week. |
Would love to see this (the OpenTracing vs OpenCensus debate is making my head spin) - is there a connector somewhere for this today? I can't seem to find anything anywhere, did the previous work just disappear? |
hi, happy to see this issue. is there possible that opencensus provide a opentraing api. a bridge. I am using opencensus, you guys are doing great! |
I'm looking for a solution as well. Although it would be great to see OC adoption instead of using bridges, when using third party stuff (eg. Traefik), we don't really have a choice. |
This will be addressed in upcoming OC - OT merging. keeping the issue open untill we haver merged api available. |
Many frameworks have already been instrumented with OpenTracing. Building an OpenTracing implementation on top of OpenCensus would give us compatibility with these frameworks.
The text was updated successfully, but these errors were encountered: