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

all: Migrate to OpenTelemetry #2877

Open
tonyhhyip opened this issue Nov 10, 2020 · 13 comments
Open

all: Migrate to OpenTelemetry #2877

tonyhhyip opened this issue Nov 10, 2020 · 13 comments

Comments

@tonyhhyip
Copy link

tonyhhyip commented Nov 10, 2020

Is your feature request related to a problem? Please describe.

OpenCensus and OpenTracing have merged to form OpenTelemetry, considering of migrate towards OpenTelemetry should be taken

Describe the solution you'd like

Make plans on migrate to OpenTelemetry from OpenCensus.

Describe alternatives you've considered

Keep using OpenCensus as most of the exporter is supported on OpenCensus

Additional context

Both Google Cloud Trace and AWS begins with support on OpenTelemetry.

@saeidakbari
Copy link

Is there any migration plan? it would look better if migration is done section by section as it will not introduce too many changes at once.

@vangent
Copy link
Contributor

vangent commented Dec 18, 2021

There are no active plans to do this.

@hsblhsn
Copy link

hsblhsn commented Jan 5, 2022

This issue is over one year old. But there is no progress. Both OpenCensus and OpenTracing have merged into OpenTelemetry! So, it's better to move to OpenTelemetry.

@tmc
Copy link

tmc commented Jul 19, 2022

@vangent can you share if anything has changed here?

@vangent
Copy link
Contributor

vangent commented Jul 21, 2022

No, nothing has changed.

#3085 exists, but hasn't seen any action in months.

I still don't understand whether this conversion would be a backwards-compatible change or not.

@vangent
Copy link
Contributor

vangent commented Sep 30, 2022

I included an announcement in todays v0.27.0 release. Let's wait a couple of weeks to see if there's any objection; if not, I'll accept the PR. We'll want to commit to updating all of the packages in this library before the next release.

@saeidakbari
Copy link

Fortunately, most packages using OpenCensus in this library have an alternative compatible version in OpenTelemetry. I don't think updating them would be challenging.

@vangent
Copy link
Contributor

vangent commented Dec 28, 2022

Update: stuck on this. @saeidakbari hasn't made any progress in a while. I cloned the PR, but it's not trivial to get tests to pass. I'd also want to update the entire library in one release, and I also haven't been able to figure out how to update the rest of the code base (e.g., blob/blob.go) to OpenTelemetry.

@Fryuni
Copy link

Fryuni commented Mar 14, 2023

Are there currently any metrics using OpenCensus or just tracing?
Go CDK exposes metrics using OpenCensus, not just tracing.

OpenTelemetry metrics specification has only recently been stabilized, and the Go implementation is still in beta. The API has been changing a lot with every release.

I'd wait to migrate until it is stable if there are metrics.

If there is only tracing, Once it is stable, I think I can help to update the tests.

@david-gang
Copy link

@vangent
Copy link
Contributor

vangent commented Sep 16, 2023

Update: no progress here and I'm not super motivated, if anyone wants to contribute the port that would be welcome.

@Fryuni
Copy link

Fryuni commented Sep 16, 2023

@vangent Do you have a branch or something locally with you in-progress work, besides #3189? The last commit there is from nearly a month before your previous update here.

You mentioned previously about problems on how to test the telemetry. Is that the only thing missing? I just recently migrated multiple of our services to OpenTelemetry v1, in multiple langs, so I had to figure those out already.

@vangent
Copy link
Contributor

vangent commented Sep 17, 2023

Nope, that pull request is all I have. A contributor did some work to convert the server package, it looked OK at first glance but I didn't look at it closely. I tried the other packages (which don't use OpenCensus much) but it wasn't obvious to me what to do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants