-
Notifications
You must be signed in to change notification settings - Fork 582
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
Do not publish otelzerolog as it is not handling core functionality #5969
Comments
Seeing both those issues have been open for 2+ years, we can't expect a quick resolution. I am a codeowner of that bridge because I helped @AkhigbeEromo set up the bridge. But I don't have any vested interest in that bridge, so I'm not particularly keen on owning that separate repository. |
I will like to take it |
@AkhigbeEromo, I assigned it to you. Please create a PR that removes the |
@pellared is going to create a PR to just remove this. The clone of this can reference the history of the repository to make the copy. |
@AkhigbeEromo I created #5996 as it is blocking the next release of OTel Go. Still, feel free to move the code to a fork and add it to the OTel Registry. Please make sure to keep the "license header" in the files to not violate the licensing. |
Hello @pellared |
The reverse of open-telemetry/opentelemetry.io#4932 |
I have the code on a branch of my fork of the repo, |
Calling it "fork" was not precise. I mean move this module to a separate repository e.g. https://github.com/AkhigbeEromo/otelzerolog. After you move the code, (configure some build pipeline), and publish a release, you can add it to the registry by summitting a PR in https://github.com/open-telemetry/opentelemetry.io |
This Issue originates from a SIG discussion around #5963 (comment) on 2024-08-01.
With
otelzerolog
bridge not being able transform fields into attribute nor handle context correlation (because of rs/zerolog#493), we do not see value in publishing (and even maintaining)otelzerolog.
Can we move this module to a separate repository until rs/zerolog#493 is solved?
CC @dmathieu @AkhigbeEromo
The text was updated successfully, but these errors were encountered: