microservices-demo icon indicating copy to clipboard operation
microservices-demo copied to clipboard

OpenTelemetry Support

Open kshcherban opened this issue 3 years ago • 11 comments

Please advise do you plan to migrate from OpenCensus to OpenTelemetry? I've seen some discussion around it in #109 but there's no track of any progress.

kshcherban avatar Oct 29 '20 14:10 kshcherban

cc @simonz130 do you know of any progress?

ahmetb avatar Oct 29 '20 18:10 ahmetb

We hope to migrate this app from OpenCensus to OpenTelemetry in the April-May 2021 timeframe, thanks for your patience.

askmeegs avatar Mar 05 '21 17:03 askmeegs

Update: after discussing with our team, we have decided to keep the upstream OnlineBoutique as a reference for OpenCensus, until its eventual deprecation.

The Cloud Ops Sandbox provides an OnlineBoutique + OpenTelemetry reference. You are also welcome to fork this repo and replace OpenCensus with OpenTelemetry.

askmeegs avatar Mar 23 '21 13:03 askmeegs

Checking in on this issue, do you expect to update to Otel still? Thanks.

jkowall avatar Jun 11 '21 15:06 jkowall

Hello all, if someone else comes across this ticket, I've forked and instrumented the Online Boutique with OpenTelemetry: https://github.com/julianocosta89/opentelemetry-microservices-demo

I've added an OpenTelemetry Collector and Jaeger to it as well.

I've also removed the OpenCensus and Cloud Operations (Stackdriver). All the changes are documented in this PR: https://github.com/julianocosta89/opentelemetry-microservices-demo/pull/1

julianocosta89 avatar Jan 31 '22 21:01 julianocosta89

We did the same, here is the fork with lots of Otel added. https://github.com/logzio/microservices-demo

jkowall avatar Feb 01 '22 18:02 jkowall

Just adding this new CNCF blog article for reference too: https://www.cncf.io/blog/2022/04/04/learn-opentelemetry-tracing-with-this-lightweight-microservices-demo/.

mathieu-benoit avatar Apr 04 '22 16:04 mathieu-benoit

Bumping this to p3, but we should consider making the switch (cc: @NimJay)

bourgeoisor avatar Jun 21 '22 21:06 bourgeoisor

Is it something you could implement during one of your rotation @arbrown or @minherz?

mathieu-benoit avatar Aug 31 '22 12:08 mathieu-benoit

@mathieu-benoit the Otel community is working on a demo that was based on the hipster shop: https://github.com/open-telemetry/opentelemetry-demo

julianocosta89 avatar Aug 31 '22 15:08 julianocosta89

Wow, thanks @julianocosta89, the idea was already to leverage the content you previously shared with us: https://github.com/GoogleCloudPlatform/microservices-demo/issues/422#issuecomment-1026247664. We will now make sure that we are leveraging the latest and greatest from https://github.com/open-telemetry/opentelemetry-demo. Thanks for sharing, much appreciated! CC: @arbrown and @minherz

mathieu-benoit avatar Aug 31 '22 15:08 mathieu-benoit

As an update:

As of #1180, we have a kustomize module for deploying the OpenTelemetry Collector and turning on distributed tracing in the microservices. Expect basic support for custom metrics and managed Prometheus this year.

arbrown avatar Nov 30 '22 23:11 arbrown