Skip to content

[NBS] Add IDeviceStatObserver to notify VolumeActor on device status changes#5565

Open
zavik001 wants to merge 9 commits intoydb-platform:mainfrom
zavik001:users/zavik001/device-statistics-class
Open

[NBS] Add IDeviceStatObserver to notify VolumeActor on device status changes#5565
zavik001 wants to merge 9 commits intoydb-platform:mainfrom
zavik001:users/zavik001/device-statistics-class

Conversation

@zavik001
Copy link
Copy Markdown

@zavik001 zavik001 commented Mar 24, 2026 •
edited by drbasic
Loading

#3727

Introduced IDeviceStatObserver interface to TDeviceStat so that
TNonreplicatedPartitionActor gets notified when a device becomes
broken or recovers, and forwards these events to VolumeActor.

@zavik001
[NBS] Convert TDeviceStat from struct to class
6a27fde 
  Preparatory step for introducing IDeviceStatObserver pattern.
  No behavioral changes.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 24, 2026

Hi! Thank you for contributing!
The tests on this PR will run after a maintainer adds an ok-to-test label to this PR manually. Thank you for your patience!

@zavik001 zavik001 closed this Mar 24, 2026
@zavik001 zavik001 reopened this Mar 24, 2026
@drbasic drbasic added ok-to-test Label to approve test launch for external members large-tests Launch large tests for PR blockstore Add this label to run only cloud/blockstore build and tests on PR labels Mar 24, 2026
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Mar 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 24, 2026

Note

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

🟢 linux-x86_64-relwithdebinfo target: cloud/blockstore/ (test time: 11100s): all tests PASSED for commit b06d85a.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
6396 6395 0 0 0 1 0

@zavik001 zavik001 changed the title [NBS] Convert TDeviceStat from struct to class [NBS] Add IDeviceStatObserver to notify VolumeActor on device status changes Mar 25, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 26, 2026 •
edited
Loading

Note

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

🔴 linux-x86_64-relwithdebinfo target: cloud/blockstore/ (test time: 11659s): some tests FAILED for commit 3fd0378.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
6402 6393 0 8 0 1 0

🔴 linux-x86_64-relwithdebinfo target: cloud/blockstore/ (test time: 47s): some tests FAILED for commit 3fd0378.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
16 8 0 8 0 0 0

🔴 linux-x86_64-relwithdebinfo target: cloud/blockstore/ (test time: 45s): some tests FAILED for commit 3fd0378.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
16 8 0 8 0 0 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 27, 2026 •
edited
Loading

Note

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

🔴 linux-x86_64-relwithdebinfo target: cloud/blockstore/ (test time: 11473s): some tests FAILED for commit c756216.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
6402 6400 0 1 0 1 0

🟢 linux-x86_64-relwithdebinfo target: cloud/blockstore/ (test time: 86s): all tests PASSED for commit c756216.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
2 2 0 0 0 0 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@drbasic drbasic drbasic approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

blockstore Add this label to run only cloud/blockstore 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.

2 participants

@zavik001 @drbasic