Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

spring-cloud-sleuth Adapter #5

Open
devinsba opened this issue Jan 12, 2017 · 6 comments
Open

spring-cloud-sleuth Adapter #5

devinsba opened this issue Jan 12, 2017 · 6 comments

Comments

@devinsba
Copy link

I have created an API adapter layer that allows OpenTracing instrumentation to be used with the spring-cloud-sleuth tracing platform.

My WIP repo is here: https://github.com/DealerDotCom/sleuth-opentracing

Would love to contribute this to OpenTracing once I have tests and a spring-boot starter created

@bhs
Copy link
Contributor

bhs commented Jan 12, 2017

@devinsba sounds great. I would be happy to shepherd this into opentracing-contrib as discussed on Gitter. Let me know how I can help.

First question: we should figure out a repo name. Maybe opentracing-contrib/java-spring-sleuth-adapter? Or just .../java-spring-sleuth?

@devinsba
Copy link
Author

java-spring-sleuth-adapter sounds better to me.

My focus in the near term will be getting tests around things and then pushing into the new repo whenever you are able to create it.

Getting ahead of this a bit: Is there a standard release process for the contrib projects? Or should I try to follow the OT/Zipkin release process

@bhs
Copy link
Contributor

bhs commented Jan 12, 2017

@devinsba the TravisCI integration that others have been using is a little annoying to set up (esp the first time) but is a wonderful thing after that initial pain. So that would be my vote.

I am thinking about creating a non-human user for opentracing maven artifacts... that way we wouldn't be using the individual credentials of lots of different people (nor would they all need write access to the maven components, etc). But I don't think I have time to set that up in the very near future.

@cmoulliard
Copy link

What is the status about this request ?

@devinsba
Copy link
Author

devinsba commented Jun 7, 2017

I put things on hold while the Context API was being worked out. I will try to update soon, but will definitely get to it when 1.0 is cut

@tedsuo
Copy link

tedsuo commented Jun 12, 2017

Thanks @devinsba we're definitely interested in an adapter, though my understanding is that what OpenTracing-based tracers can do with it is limited, due to things such as Spans being created in Sleuth, outside of the Tracer's control, plus what you have mentioned in your repo.

So, I'm concerned as to how useful such a bridge can actually be for production tracing. But I would rather have a well made bridge that clearly illuminates what is missing or tricky, even if it is not sufficient for full OT tracing. Also, if you think that production-quality tracing can be done with a Sleuth/OT bridge in it's current state, I'd like to hear the case for it so that I can better understand the current state of things.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants