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

Resize the shadow disk when the size of the source disk changes #2152

Merged
merged 2 commits into from
Sep 30, 2024

Conversation

drbasic
Copy link
Collaborator

@drbasic drbasic commented Sep 27, 2024

Когда происходит изменение диска нужно изменить и размер теневого диска тоже, чтобы если в этот момент идет миграция, то записи пользователей могли проходить, а не падать с ошибкой E_ARGUMENT, что приводит к поломке NRD и залипанию mirror-диска.
Т.е. фикс чинит проблему, когда пользователь изменил размер диска в процессе создания снимка.

@drbasic drbasic added the blockstore Add this label to run only cloud/blockstore build and tests on PR label Sep 27, 2024
Copy link
Contributor

Note

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

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

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

qkrorlqr
qkrorlqr previously approved these changes Sep 27, 2024
Copy link
Contributor

Note

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

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3289 3288 0 1 0 0

Copy link
Contributor

Note

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

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3289 3288 0 1 0 0

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

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

@drbasic drbasic merged commit 099fbb8 into main Sep 30, 2024
11 of 13 checks passed
@drbasic drbasic deleted the users/drbasic/reallocate-shadow-disk branch September 30, 2024 11:43
drbasic added a commit that referenced this pull request Oct 1, 2024
* Resize the shadow disk when the size of the source disk changes

* Rename method
drbasic added a commit that referenced this pull request Oct 2, 2024
* Handle migration partition non-retriable errors separately (#2142)

* Handle migration partition non-retriable errors separately

* Tests

* Fix review issues

* fixed use-after-free in partition_nonrepl uts (#2184)

* Fix wrong response type usage (#2150)

* Resize the shadow disk when the size of the source disk changes (#2152)

* Resize the shadow disk when the size of the source disk changes

* Rename method

---------

Co-authored-by: Andrei Strelkovskii <thinkingwhat@yandex.ru>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blockstore Add this label to run only cloud/blockstore build and tests on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants