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

Integration with gRPC #305

Closed
testark opened this issue Jun 16, 2016 · 10 comments · Fixed by #829
Closed

Integration with gRPC #305

testark opened this issue Jun 16, 2016 · 10 comments · Fixed by #829
Assignees
Milestone

Comments

@testark
Copy link

testark commented Jun 16, 2016

Is instrumentation for gRPC in your roadmap for anytime soon?

@marcingrzejszczak
Copy link
Contributor

Nope - it's not ATM.

@ghost
Copy link

ghost commented Jan 20, 2017

+1 for still implementing it :)

@sdavids
Copy link

sdavids commented Aug 11, 2017

Why is this closed?

@marcingrzejszczak
Copy link
Contributor

Most likely by accident @sdavids

@thekalinga
Copy link

thekalinga commented Jan 7, 2018

Is there a chance this would be fixed anytime soon?

gRPC is much more efficient as it uses HTTP 2.0, plus we can leverage new service mesh frameworks like Envoy that are far more superior & feature rich than Hystrix (netflix family ingeneral)

@codefromthecrypt
Copy link
Contributor

#711 will resolve this indirectly or directly as brave supports grpc

@marcingrzejszczak marcingrzejszczak added this to the 2.0.0.M6 milestone Jan 19, 2018
@marcingrzejszczak marcingrzejszczak self-assigned this Jan 19, 2018
marcingrzejszczak added a commit that referenced this issue Jan 19, 2018
with this pull request we have rewritten the whole Sleuth internals to use Brave. That way we can leverage all the functionalities & instrumentations that Brave already has (https://github.com/openzipkin/brave/tree/master/instrumentation).

Migration guide is available here: https://github.com/spring-cloud/spring-cloud-sleuth/wiki/Spring-Cloud-Sleuth-2.0-Migration-Guide

fixes #711 - Brave instrumentation
fixes #92 - we move to Brave's Sampler
fixes #143 - Brave is capable of passing context
fixes #255 - we've moved away from Zipkin Stream server
fixes #305 - Brave has GRPC instrumentation (https://github.com/openzipkin/brave/tree/master/instrumentation/grpc)
fixes #459 - Brave (openzipkin/brave#510) & Zipkin (openzipkin/zipkin#1754) will deal with the AWS XRay instrumentation
fixes #577 - Messaging instrumentation has been rewritten
@codefromthecrypt
Copy link
Contributor

Let's see about more out-of-box integration

you can see tests for client and server interceptors here
https://github.com/openzipkin/brave/blob/master/instrumentation/grpc/src/test/java/brave/grpc/ITTracingClientInterceptor.java
https://github.com/openzipkin/brave/blob/master/instrumentation/grpc/src/test/java/brave/grpc/ITTracingServerInterceptor.java

there are probably some example spring boot + grpc projects out there.. key is doing the interceptor add-on conditionally when building server or client channel

@marcingrzejszczak marcingrzejszczak modified the milestones: 2.0.0.M6, 2.1.0 May 8, 2018
@tkvangorder
Copy link
Contributor

tkvangorder commented May 10, 2018

Since brave has client and server side interceptors for gRpc (thanks for the pointer @adriancole ), It is actually pretty straighforward to instrument gRcp calls by combining Spring Cloud Sleuth 2.x with the third-party boot library grpc-spring-boot-starter. I have setup a small sample project that demonstrates how this can be done here: https://github.com/tkvangorder/sleuth-grpc-sample

@spring-projects-issues
Copy link

Closing due to age of the question. If you would like us to look at this issue, please comment and we will look at re-opening the issue.

@ghost
Copy link

ghost commented Dec 5, 2018

Thanks for still implementing it!

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

Successfully merging a pull request may close this issue.

7 participants