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

YDB client logger does not write fields from context #487

Open
gy2411 opened this issue Feb 19, 2024 · 0 comments
Open

YDB client logger does not write fields from context #487

gy2411 opened this issue Feb 19, 2024 · 0 comments
Assignees
Labels
bug Something isn't working disk_manager Add this label to run only cloud/disk_manager build and tests on PR

Comments

@gy2411
Copy link
Collaborator

gy2411 commented Feb 19, 2024

YDB client uses its own logger, which calls methods of underlying logger (code). It should write fields from context, but it does not. In particular, it does not write syslog identifier.

Proposed solution:

  1. Make possible to add arbitrariy logging fields to context.
  2. In the YDB client logger, add all necessary fields to the context and then call functions from the interface.
@gy2411 gy2411 added bug Something isn't working disk_manager Add this label to run only cloud/disk_manager build and tests on PR labels Feb 19, 2024
@gy2411 gy2411 self-assigned this Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working disk_manager Add this label to run only cloud/disk_manager build and tests on PR
Projects
None yet
Development

No branches or pull requests

1 participant