fix(vm): fix some volume migration bugs #1498
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
yaroslavborbat
requested review from
hardcoretime and
danilrwx
as code owners
Isteb4k
requested review from
universal-itengineer and
nevermarine
as code owners
Isteb4k
force-pushed
the
fix/vm/volume-migration
branch
from
68b224f to
18d338a
Compare
yaroslavborbat
force-pushed
the
fix/vm/volume-migration
branch
2 times, most recently
from
735917f to
2acfd13
Compare
yaroslavborbat
force-pushed
the
fix/vm/volume-migration
branch
from
010db18 to
dadefc7
Compare
Isteb4k
approved these changes
Sep 25, 2025
|
/backport 1.1 |
github-actions bot
pushed a commit
that referenced
this pull request
Sep 25, 2025
This PR fixes several critical bugs related to volume migration in VM operations: 1. Fixed Migrating Condition and Generation Sync Problem: Incorrect handling of the Migrating condition and lack of proper synchronization. Solution: Added proper synchronization logic and correct setting of observedGeneration to ensure state consistency. 2. Blocked RWO Volume Migration with Immediate Failure Problem: Attempting to migrate VM with RWO (ReadWriteOnce) volumes was causing undefined behavior. Solution: VMOP now immediately fails the migration operation when RWO volumes are detected, as this configuration is not supported. Added e2e test. 3. Fixed Pending State Hang and Volume Sync on KVVM Problem: VM operations could hang indefinitely in Pending state due to volume synchronization issues on KVVM. Solution: Implemented proper volume synchronization logic to prevent hanging and ensure smooth state transitions. 4. Prevented Unplanned Restarts from Unexpected State Problem: Presence of unexpected state could trigger unplanned VM restarts. Solution: Added validation and handling logic to prevent restarts caused by unexpected state conditions. Signed-off-by: Yaroslav Borbat <yaroslav.borbat@flant.com>
deckhouse-BOaTswain
added a commit
that referenced
this pull request
Sep 25, 2025
fix(vm): fix some volume migration bugs (#1498) This PR fixes several critical bugs related to volume migration in VM operations: 1. Fixed Migrating Condition and Generation Sync Problem: Incorrect handling of the Migrating condition and lack of proper synchronization. Solution: Added proper synchronization logic and correct setting of observedGeneration to ensure state consistency. 2. Blocked RWO Volume Migration with Immediate Failure Problem: Attempting to migrate VM with RWO (ReadWriteOnce) volumes was causing undefined behavior. Solution: VMOP now immediately fails the migration operation when RWO volumes are detected, as this configuration is not supported. Added e2e test. 3. Fixed Pending State Hang and Volume Sync on KVVM Problem: VM operations could hang indefinitely in Pending state due to volume synchronization issues on KVVM. Solution: Implemented proper volume synchronization logic to prevent hanging and ensure smooth state transitions. 4. Prevented Unplanned Restarts from Unexpected State Problem: Presence of unexpected state could trigger unplanned VM restarts. Solution: Added validation and handling logic to prevent restarts caused by unexpected state conditions. Signed-off-by: Yaroslav Borbat <yaroslav.borbat@flant.com> Co-authored-by: Yaroslav Borbat <86148689+yaroslavborbat@users.noreply.github.com>
|
Cherry pick PR 1505 to the branch release-1.1 successful! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR fixes several critical bugs related to volume migration in VM operations:
1. Fixed Migrating Condition and Generation Sync
Migratingcondition and lack of proper synchronization.observedGenerationto ensure state consistency.2. Blocked RWO Volume Migration with Immediate Failure
3. Fixed Pending State Hang and Volume Sync on KVVM
Pendingstate due to volume synchronization issues on KVVM.4. Prevented Unplanned Restarts from Unexpected State
Why do we need it, and what problem does it solve?
What is the expected result?
Checklist