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

Migrate from opencensus library #29867

Open
7 of 15 tasks
codeboten opened this issue Dec 13, 2023 · 5 comments
Open
7 of 15 tasks

Migrate from opencensus library #29867

codeboten opened this issue Dec 13, 2023 · 5 comments
Assignees
Labels
collector-telemetry never stale Issues marked with this label will be never staled and automatically removed

Comments

@codeboten
Copy link
Contributor

codeboten commented Dec 13, 2023

Component(s)

No response

Describe the issue you're reporting

This is an encompassing issue to capture all the components using the opencensus library to generate metrics today. Migrating them to use OpenTelemetry should be a fairly quick process, and once telemetry.useOtelForInternalMetrics is stable, we should be able to move all components on this list to emit metrics with the OTel SDK

@atoulme
Copy link
Contributor

atoulme commented Dec 14, 2023

How can I help with exporter/splunkhec? What is the equivalent API to adopt?

@codeboten
Copy link
Contributor Author

How can I help with exporter/splunkhec? What is the equivalent API to adopt?

@atoulme i'm trying to put an example together, unfortunately, the two components i started with didn't work out, one was using census without needing it (see the statsd PR) and the other, well i ended up not changing it (healthcheck extension) 😆

Will comment on here once i have an example 👍🏻

codeboten pushed a commit that referenced this issue Dec 15, 2023
Most of the methods in the Reporter interface were not used beyond being
called in a test. Removing them, which removes the dependency on
opencensus.

related to #29867

Signed-off-by: Alex Boten <aboten@lightstep.com>
@codeboten
Copy link
Contributor Author

@atoulme here's an example of migrating from opencensus to otel: #29917

bogdandrutu pushed a commit that referenced this issue Dec 15, 2023
This updates the servicegraph processor to emit telemetry using
OpenTelemetry instead of OpenCensus.

Related #29867

---------

Signed-off-by: Alex Boten <aboten@lightstep.com>
nslaughter pushed a commit to nslaughter/opentelemetry-collector-contrib that referenced this issue Dec 27, 2023
Most of the methods in the Reporter interface were not used beyond being
called in a test. Removing them, which removes the dependency on
opencensus.

related to open-telemetry#29867

Signed-off-by: Alex Boten <aboten@lightstep.com>
nslaughter pushed a commit to nslaughter/opentelemetry-collector-contrib that referenced this issue Dec 27, 2023
…etry#29917)

This updates the servicegraph processor to emit telemetry using
OpenTelemetry instead of OpenCensus.

Related open-telemetry#29867

---------

Signed-off-by: Alex Boten <aboten@lightstep.com>
nslaughter pushed a commit to nslaughter/opentelemetry-collector-contrib that referenced this issue Dec 27, 2023
Most of the methods in the Reporter interface were not used beyond being
called in a test. Removing them, which removes the dependency on
opencensus.

related to open-telemetry#29867

Signed-off-by: Alex Boten <aboten@lightstep.com>
nslaughter pushed a commit to nslaughter/opentelemetry-collector-contrib that referenced this issue Dec 27, 2023
…etry#29917)

This updates the servicegraph processor to emit telemetry using
OpenTelemetry instead of OpenCensus.

Related open-telemetry#29867

---------

Signed-off-by: Alex Boten <aboten@lightstep.com>
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this issue Jan 10, 2024
Most of the methods in the Reporter interface were not used beyond being
called in a test. Removing them, which removes the dependency on
opencensus.

related to open-telemetry#29867

Signed-off-by: Alex Boten <aboten@lightstep.com>
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this issue Jan 10, 2024
…etry#29917)

This updates the servicegraph processor to emit telemetry using
OpenTelemetry instead of OpenCensus.

Related open-telemetry#29867

---------

Signed-off-by: Alex Boten <aboten@lightstep.com>
Copy link
Contributor

github-actions bot commented Apr 8, 2024

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

@sumo-drosiek
Copy link
Member

Another Issue: #32261

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
collector-telemetry never stale Issues marked with this label will be never staled and automatically removed
Projects
None yet
Development

No branches or pull requests

3 participants