Use OTLP format instead of OpenCensus for OTLP exporter

[mayurkale22]

Currently, we use OpenCensus proto based types to export traces from OpenTelemetry instrumentation to the Collector exporter. The goal is to switch to OTLP format/proto (link attached) instead of OpenCensus for OTLP exporter.

Trace Proto: https://github.com/open-telemetry/opentelemetry-proto/blob/master/opentelemetry/proto/trace/v1/trace.proto

Resource Proto: https://github.com/open-telemetry/opentelemetry-proto/blob/master/opentelemetry/proto/resource/v1/resource.proto

[mayurkale22]

@dyladan assign to you if you're working on this, otherwise will take it.

[dyladan]

I think @obecny is working on this

[dyladan]

When I offered to take this I had forgotten that he already started work. If I'm wrong here, I'm happy to do it.

[obecny]

Yea I'm almost finished, just trying to have working example ideally with docker (as previously) with https://github.com/open-telemetry/opentelemetry-collector

[dyladan]

AFAIK there are premade collector docker images on the omnition dockerhub

[obecny]

@dyladan already on chat with @mayurkale22 - however cannot make it work yet

[obecny]

but the quick question as the new proto format so far I have implemented only for node version, for web it stays as it was. Should then this be done for both versions ?

[mayurkale22]

@tigrannajaryan please confirm if this is the latest docker image with OpenTelemetry receiver support: https://hub.docker.com/r/omnition/opentelemetry-collector-contrib/tags

[obecny]

@tigrannajaryan please confirm if this is the latest docker image with OpenTelemetry receiver support: https://hub.docker.com/r/omnition/opentelemetry-collector-contrib/tags

grpc support

[mayurkale22]

but the quick question as the new proto format so far I have implemented only for node version, for web it stays as it was. Should then this be done for both versions ?

Yes, we should update for both. I believe the transformation logic is common for both Node and browser, right?

[dyladan]

We may want to avoid having to deploy grpc with the browser. In that case, we can use the new format but still transport it over http.

[tigrannajaryan]

@tigrannajaryan please confirm if this is the latest docker image with OpenTelemetry receiver support: https://hub.docker.com/r/omnition/opentelemetry-collector-contrib/tags

Confirmed. Note that this does not include OTLP receiver/exporter.

[mayurkale22]

Confirmed. Note that this does not include OTLP receiver/exporter.

@tigrannajaryan Thanks for the confirmation. May I know the timeline to get a new image with OTLP receiver/exporter support?

@obecny I think you should open your PR for reviews, for now we can rely on unit tests and run end to end test once new image is available. WDYT?