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] Issue 2732: add change device/agent state buttons #2760

Merged
merged 13 commits into from
Jan 16, 2025
Merged

[NBS] Issue 2732: add change device/agent state buttons #2760

merged 13 commits into from
Jan 16, 2025

Conversation

vladstepanyuk
Copy link
Contributor

@vladstepanyuk vladstepanyuk commented Dec 24, 2024 •
edited
Loading

#2732
Change agent state:
image
Change device state:
image

@github-actions GitHub Actions
Copy link
Contributor

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!

drbasic
drbasic previously requested changes Dec 25, 2024
View reviewed changes
cloud/blockstore/libs/storage/disk_registry/disk_registry_actor_change_agent_state.cpp Outdated Show resolved Hide resolved
cloud/blockstore/libs/storage/disk_registry/disk_registry_actor_change_agent_state.cpp Outdated Show resolved Hide resolved
cloud/blockstore/libs/storage/disk_registry/disk_registry_actor_change_agent_state.cpp Outdated Show resolved Hide resolved
cloud/blockstore/libs/storage/disk_registry/disk_registry_actor_change_agent_state.cpp Outdated Show resolved Hide resolved
cloud/blockstore/libs/storage/disk_registry/disk_registry_actor_change_device_state.cpp Outdated Show resolved Hide resolved
cloud/blockstore/libs/storage/disk_registry/disk_registry_actor_change_device_state.cpp Outdated Show resolved Hide resolved
cloud/blockstore/libs/storage/disk_registry/disk_registry_actor_change_device_state.cpp Outdated Show resolved Hide resolved
cloud/blockstore/libs/storage/disk_registry/disk_registry_actor_change_device_state.cpp Outdated Show resolved Hide resolved
@komarevtsev-d komarevtsev-d added large-tests Launch large tests for PR blockstore Add this label to run only cloud/blockstore build and tests on PR labels Jan 2, 2025
@komarevtsev-d komarevtsev-d self-requested a review January 2, 2025 07:12
@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 2, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Jan 2, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 2, 2025

Note

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

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3753 3751 0 2 0 0

@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 9, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Jan 9, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 9, 2025

Note

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

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

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

@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 10, 2025
@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 13, 2025
komarevtsev-d
komarevtsev-d previously approved these changes Jan 13, 2025
View reviewed changes
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Jan 13, 2025
@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 6dcd139.

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

@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 14, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Jan 14, 2025
@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 72b9a6e.

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

@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 15, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Jan 15, 2025
sharpeye
sharpeye previously approved these changes Jan 15, 2025
View reviewed changes
@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 15, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Jan 15, 2025
@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 01f1c8f.

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

@komarevtsev-d komarevtsev-d dismissed drbasic’s stale review January 16, 2025 05:07

Не актуально

@komarevtsev-d komarevtsev-d merged commit 93e2bcb into ydb-platform:main Jan 16, 2025
17 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

@sharpeye sharpeye sharpeye approved these changes

@drbasic drbasic Awaiting requested review from drbasic

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.
4 participants
@vladstepanyuk @drbasic @sharpeye @komarevtsev-d