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

Can't get gRPC Context Client to Server to match #434

Closed
ethanrubio opened this issue Mar 7, 2017 · 5 comments
Closed

Can't get gRPC Context Client to Server to match #434

ethanrubio opened this issue Mar 7, 2017 · 5 comments
Assignees
Labels
api: cloudtrace Issues related to the googleapis/cloud-trace-nodejs API. 🚨 This issue needs some love. triage me I really want to be triaged.

Comments

@ethanrubio
Copy link
Contributor

I'm trying to use this library to trace gRPC and Hapi requests throughout a small microservice environment.

There are three services (using a combo of gRPC and HapI) that I'm trying to log correctly in StackDriver Trace. I started all three services under the same project, but on StackDriver, I'm unable to link up the gRPC Client requests with the gRPC Server handling. However, a service with gRPC making HTTP requests to a Hapi server are correctly matched up on the web interface.

Any insights would be greatly appreciated! Thanks!

@matthewloring
Copy link
Contributor

We currently propagate distributed context using an http header meaning that distributed tracing will only work over http for now. We are designing an alternate mechanism for grpc (likely using http://www.grpc.io/docs/guides/concepts.html#metadata) but need to coordinate between different language trace agents (we want to ensure a request sent from a Node.js grpc client to a go grpc server will handle context correctly).

@kjin
Copy link
Contributor

kjin commented Mar 8, 2017

I've opened #436 to address this by writing trace context to gRPC metadata. While this field isn't standardized for other Cloud Trace clients, it should provide distributed tracing for Node.js microservices.

@kjin
Copy link
Contributor

kjin commented Mar 15, 2017

Hi @ethanrubio, we just release version 1.0.1 which includes the PR mentioned above. Please give it a try when you have the time and feel free to re-open this issue if there are any problems!

@kjin kjin closed this as completed Mar 15, 2017
@ethanrubio
Copy link
Contributor Author

ethanrubio commented Mar 17, 2017

@kjin Thanks again for adding this! Is it possible to search/filter for x-cloud-trace-context id in the web UI?

@matthewloring
Copy link
Contributor

The value of x-cloud-trace-context has the form <trace_id>/<span_id>;o=. It is possible to look up a trace from a trace id by navigating to https://console.cloud.google.com/traces/traces?project=<your_project_name>&tid=<your_trace_id>.

@google-cloud-label-sync google-cloud-label-sync bot added the api: cloudtrace Issues related to the googleapis/cloud-trace-nodejs API. label Jan 31, 2020
@yoshi-automation yoshi-automation added triage me I really want to be triaged. 🚨 This issue needs some love. labels Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: cloudtrace Issues related to the googleapis/cloud-trace-nodejs API. 🚨 This issue needs some love. triage me I really want to be triaged.
Projects
None yet
Development

No branches or pull requests

4 participants