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

cln-plugin: Switch from env_logger to tracing-subscriber #6872

Merged

Conversation

cdecker
Copy link
Member

@cdecker cdecker commented Nov 15, 2023

This switches the logging implementation from using the log-facade to using the tracing-subscriber instead. This allows us to also tap into the tracing instrumentation if desired, which was not possible with log.

Changelog-Changed cln-plugin: The logging adapter now uses tracing-subscriber allowing the tracing ecosystem to be used. No format changes.

@cdecker cdecker added this to the v24.02 milestone Nov 15, 2023
@cdecker cdecker self-assigned this Nov 15, 2023
@cdecker cdecker force-pushed the 20231115-cln-plugin-tracing-log branch from 5616a23 to 8dc9623 Compare December 5, 2023 18:25
This switches the logging implementation from using the `log`-facade
to using the `tracing-subscriber` instead. This allows us to also tap
into the tracing instrumentation if desired, which was not possible
with `log`.

Changelog-Changed cln-plugin: The logging adapter now uses tracing-subscriber allowing the `tracing` ecosystem to be used. No format changes.
@cdecker cdecker force-pushed the 20231115-cln-plugin-tracing-log branch from 8dc9623 to 13aaf76 Compare January 16, 2024 17:20
@cdecker cdecker enabled auto-merge (rebase) January 16, 2024 17:20
@cdecker cdecker merged commit 60e1532 into ElementsProject:master Jan 16, 2024
9 of 13 checks passed
@cdecker cdecker deleted the 20231115-cln-plugin-tracing-log branch January 16, 2024 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant