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.

Sign up for GitHub

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

Jump to bottom

[NBS] Increase logging verbosity: include group ID #2952

Merged
merged 1 commit into from
Jan 30, 2025
Merged

[NBS] Increase logging verbosity: include group ID #2952

merged 1 commit into from
Jan 30, 2025
+12 −4

Conversation

ya-ksgamora
Copy link
Collaborator

@ya-ksgamora ya-ksgamora commented Jan 30, 2025 •
edited
Loading

Increase logging verbosity: include group ID in (Read|Write)Blob logs

This is useful when multiple partition tablets fail due to corresponding groups failure because it allows to indicate number of unique failing groups

@ya-ksgamora ya-ksgamora added the blockstore Add this label to run only cloud/blockstore build and tests on PR label Jan 30, 2025
drbasic
drbasic previously approved these changes Jan 30, 2025
View reviewed changes
@ya-ksgamora ya-ksgamora force-pushed the users/ya-ksgamora/NBS-5944 branch from 3b73ae8 to 600a1a5 Compare January 30, 2025 13:54
SvartMetal
SvartMetal previously approved these changes Jan 30, 2025
View reviewed changes
drbasic
drbasic previously approved these changes Jan 30, 2025
View reviewed changes
@ya-ksgamora ya-ksgamora dismissed stale reviews from drbasic and SvartMetal via 8ba3a79 January 30, 2025 14:03
@ya-ksgamora ya-ksgamora force-pushed the users/ya-ksgamora/NBS-5944 branch from 600a1a5 to 8ba3a79 Compare January 30, 2025 14:03
@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 8ba3a79.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3609 3609 0 0 0 0

@ya-ksgamora ya-ksgamora merged commit 53e512a into main Jan 30, 2025
12 of 15 checks passed
@ya-ksgamora ya-ksgamora deleted the users/ya-ksgamora/NBS-5944 branch January 30, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@drbasic drbasic

@SvartMetal SvartMetal

@BarkovBG BarkovBG

Assignees
No one assigned
Labels
blockstore Add this label to run only cloud/blockstore build and tests on PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@ya-ksgamora @drbasic @SvartMetal @BarkovBG