Skip to content

Drop opencensus dependency #1016

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.

Sign up for GitHub

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

Jump to bottom
Closed
olegrok opened this issue Feb 4, 2025 · 4 comments · Fixed by #1045
Closed

Drop opencensus dependency #1016

olegrok opened this issue Feb 4, 2025 · 4 comments · Fixed by #1045

Comments

@olegrok
Copy link

olegrok commented Feb 4, 2025

DHT uses go.opencensus.io package, which has been archived since 2023 and is no longer recommended for use. Additionally, it cannot be downloaded without the Go proxy due to an expired certificate on their site.

I suggest transitioning to OpenTelemetry, as it is now the standard in the industry and is recommended by OpenCensus itself.
@guillaumemichel
Copy link
Contributor

Makes sense! @olegrok do you want to make the PR?

@AvyChanna
Copy link
Contributor

I have a branch with otel changes. Can raise a PR for this

@guillaumemichel
Copy link
Contributor

@AvyChanna yes, thanks a lot, I would be happy to review it.

@AvyChanna
Copy link
Contributor

Raised #1045

@guillaumemichel guillaumemichel linked a pull request Feb 21, 2025 that will close this issue
feat: use OTEL for metrics (removes opencensus) #1045
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: use OTEL for metrics (removes opencensus) AvyChanna/go-libp2p-kad-dht
3 participants
@olegrok @guillaumemichel @AvyChanna