-
Notifications
You must be signed in to change notification settings - Fork 53
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
feat!(opentelemetry): migrate from opencensus to opentelemetry #266
base: master
Are you sure you want to change the base?
Conversation
Since we want to guarantee backward compatibility with OpenCensus, I tried to use the OpenCensus bridge module. However, an open issue hinders me. I'm trying to find a workaround. |
As of now, it's not possible to define boundaries for histograms at the instrument level with opentelemetry-go. |
|
|
3e645a2
to
1ca1f56
Compare
… using Opentelemetry
1ca1f56
to
df3b13d
Compare
OpenTelemetry module now has its own repository: https://github.com/i-love-flamingo/opentelemetry We can not merge this PR right now, because it contains breaking changes. With all the traces and metrics moved to the OpenTelemetry API, they are no longer exposed for the opencensus module. So we have to keep this PR around, until we decide to go for a flamingo v4 that accepts breaking changes. I created another PR to mark the opencensus module as deprecated, see #364 |
What?
Flamingo uses Opencensus to collect application metrics and traces and transfer the data to a backend of the user's choice.
This pull request aims to replace Opencensus with Opentelemetry.