-
Notifications
You must be signed in to change notification settings - Fork 505
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
Add open telemetry severity hook integration for zerolog #5782
base: main
Are you sure you want to change the base?
Add open telemetry severity hook integration for zerolog #5782
Conversation
Hello @MrAlias, hope this is okay for a start |
Also, do i need to make a changelog for each of the PRs concerning the zerolog bridge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would recommend splitting this even more, with a PR that sets up the boilerplate structs/interfaces that will be used in the end, without their implementations.
Much like what was done in #5449
Thank you @dmathieu, I know what to do now. I will split up and send in another PR |
This PR introduces the
otelzerolog
package, which provides aSeverityHook
to bridge the logging capabilities of Zerolog with OpenTelemetry. The following key features are included:Changes
config
struct to handle configuration options.Option
interface and related functions (WithVersion
,WithSchemaURL
,WithLoggerProvider
).NewSeverityHook
function to initialize aSeverityHook
.SeverityHook
struct to perform the logging bridge functionality.Part of #5405