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

OpenTelemetry Support #422

Closed
kshcherban opened this issue Oct 29, 2020 · 12 comments
Closed

OpenTelemetry Support #422

kshcherban opened this issue Oct 29, 2020 · 12 comments
Assignees
Labels
area/monitoring priority: p3 Desirable enhancement or fix. May not be included in next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@kshcherban
Copy link

kshcherban commented Oct 29, 2020

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.

@ahmetb
Copy link
Contributor

ahmetb commented Oct 29, 2020

cc @simonz130 do you know of any progress?

@askmeegs
Copy link
Contributor

askmeegs commented Mar 5, 2021

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

@askmeegs
Copy link
Contributor

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 askmeegs added priority: p3 Desirable enhancement or fix. May not be included in next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. and removed priority/icebox what/feature labels Apr 8, 2021
@jkowall
Copy link

jkowall commented Jun 11, 2021

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

@NimJay NimJay changed the title Opentelemetry support OpenTelemetry Support Nov 22, 2021
@julianocosta89
Copy link

julianocosta89 commented Jan 31, 2022

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:
julianocosta89#1

@jkowall
Copy link

jkowall commented Feb 1, 2022

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

@NimJay NimJay added priority: p2 Moderately-important priority. Fix may not be included in next release. and removed priority: p3 Desirable enhancement or fix. May not be included in next release. labels Apr 1, 2022
@mathieu-benoit
Copy link
Contributor

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/.

@bourgeoisor bourgeoisor added priority: p3 Desirable enhancement or fix. May not be included in next release. and removed priority: p2 Moderately-important priority. Fix may not be included in next release. labels Jun 21, 2022
@bourgeoisor
Copy link
Member

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

@mathieu-benoit
Copy link
Contributor

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

@julianocosta89
Copy link

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

@mathieu-benoit
Copy link
Contributor

Wow, thanks @julianocosta89, the idea was already to leverage the content you previously shared with us: #422 (comment). 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

@arbrown
Copy link
Contributor

arbrown commented Nov 30, 2022

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 arbrown closed this as completed Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/monitoring priority: p3 Desirable enhancement or fix. May not be included in next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Development

No branches or pull requests

10 participants