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

[CT-1444] Structured logging: combine node_info events with different logging levels #6173

Closed
gshank opened this issue Oct 28, 2022 · 0 comments · Fixed by #6174
Closed

[CT-1444] Structured logging: combine node_info events with different logging levels #6173

gshank opened this issue Oct 28, 2022 · 0 comments · Fixed by #6174
Labels
Milestone

Comments

@gshank
Copy link
Contributor

gshank commented Oct 28, 2022

Prior to this ticket a number of logging events had different event classes depending on the "level" of the logging. This makes it more difficult for consumers of the logs since they would have to subscribe to all versions of the events.

This ticket collapses the multiple events into single events.

This is spun off as part of the work being done in #5663 (CT-1048). Additional work will be done in that ticket to create proto messages for various nodes and node configs.

@github-actions github-actions bot changed the title Structured logging: combine node_info events with different logging levels [CT-1444] Structured logging: combine node_info events with different logging levels Oct 28, 2022
@jtcohen6 jtcohen6 added this to the v1.4 milestone Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants