Skip to content

Log improvements for TVolumeProxyActor #3728

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
Merged
merged 3 commits into from
Jun 24, 2025
Merged

Log improvements for TVolumeProxyActor #3728

merged 3 commits into from
Jun 24, 2025

Conversation

drbasic
Copy link
Collaborator

@drbasic drbasic commented Jun 21, 2025 •
edited
Loading

  1. Добавил запись в лог при установке соединения (когда мы узнали TabletId)
  2. Добавил запись в лог при пере-установке соединения
  3. теперь видно что это сообщения от NBS-2
2025-06-21T14:29:49.754996Z :BLOCKSTORE_VOLUME_PROXY INFO: [vp:72075186224037890 d:nrd1 pg:1 t:1.640ms] Volume with path "/Root/NBS/_2EB/nrd1" resolved
2025-06-21T14:30:49.755731Z :BLOCKSTORE_VOLUME_PROXY INFO: [vp:72075186224037890 d:nrd1 pg:1 t:60.000s] Remove connection

@drbasic drbasic added large-tests Launch large tests for PR disk_manager Add this label to run only cloud/disk_manager build and tests on PR blockstore Add this label to run only cloud/blockstore build and tests on PR labels Jun 21, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jun 21, 2025 •
edited
Loading

Note

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

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 59764bd.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6542 6540 0 1 1 0

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 59764bd.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
65 64 0 1 0 0

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 59764bd.

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

@drbasic drbasic added the recheck Add this label to relaunch checks, it will be automatically removed label Jun 22, 2025
@github-actions github-actions bot removed the recheck Add this label to relaunch checks, it will be automatically removed label Jun 22, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jun 22, 2025 •
edited
Loading

Note

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

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 59764bd.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6542 6539 0 2 1 0

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 59764bd.

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

komarevtsev-d
komarevtsev-d previously approved these changes Jun 23, 2025
View reviewed changes
SvartMetal
SvartMetal reviewed Jun 23, 2025
View reviewed changes
SvartMetal
SvartMetal reviewed Jun 23, 2025
View reviewed changes
SvartMetal
SvartMetal reviewed Jun 23, 2025
View reviewed changes
@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 105981b.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6542 6541 0 0 1 0

@drbasic drbasic requested a review from SvartMetal June 23, 2025 15:13
@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 2a4ee15.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6542 6541 0 0 1 0

@drbasic drbasic merged commit 4428dde into main Jun 24, 2025
11 of 12 checks passed
@drbasic drbasic deleted the users/drbasic/volume-proxy-logs-improvement branch June 24, 2025 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SvartMetal SvartMetal SvartMetal approved these changes

@komarevtsev-d komarevtsev-d komarevtsev-d approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@drbasic @SvartMetal @komarevtsev-d