opencensus-service icon indicating copy to clipboard operation
opencensus-service copied to clipboard

collector: jaeger exporter dropping root spans

Open tcolgate opened this issue 5 years ago • 6 comments

I'm testing opencensus service (naster ) with: jaeger hotrod demo -> ocagent (jaeger-compact-thrift) -> occollector -> jaeger-collector

In this configuration, all the intial root spans from the first web request are being dropped by occollector. They appear to contain a nil Link parent link. The entire batch of span the root span is in gets thrown away. I've dumped the links in spans from the agent, and it certainly looks like the jaeger client in hotrod is actually sending this nil link, ocagent is faithfully converting them.

tcolgate avatar Jun 27 '19 14:06 tcolgate

#594 resolves the export, but extra work may be needed, and the links /could/ be dropped at the agent too.

tcolgate avatar Jun 27 '19 14:06 tcolgate

I stuck a printf to dump the links directly from the client-go in hotrod that dumps the thrift spans, and the span is definitely coming from there. The agent appears to be doing the right thing.

tcolgate avatar Jun 27 '19 15:06 tcolgate

As you see the current implementation is very strict it doesn't accept zero or empty IDs. Did your dump show how the id was set?

pjanotti avatar Jun 27 '19 16:06 pjanotti

It was coming out of the Jaeger reference app, so, from what I can tell,real clients in the wild are doing this. It may be more correct then to strip it at the agent?

tcolgate avatar Jun 27 '19 17:06 tcolgate

Probably yes, but I'm trying to understand why we didn't see this before. For instance in the past we had similar issues but because the 8 byte ids were not being correctly converted to 16 byte ids in OC.

pjanotti avatar Jun 27 '19 17:06 pjanotti

@pjanotti I think this may be the cause: https://github.com/jaegertracing/jaeger-client-go/issues/242

tcolgate avatar Jun 28 '19 07:06 tcolgate

This repository is now archived, and is succeeded by https://github.com/open-telemetry/opentelemetry-collector.

dashpole avatar Nov 07 '22 15:11 dashpole