-
Notifications
You must be signed in to change notification settings - Fork 1.3k
kvm: fix live vm migration between local storage pools #7945
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
Conversation
|
@blueorangutan package |
|
@weizhouapache a [SF] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
yadvr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM tested this on a 3-host Ubuntu 22.04 kvm cluster and with this patch I'm no longer facing errors when live migrating VMs with local storage. Surpringly, this happens in my env when disk sizes are large (>20GB).
harikrishna-patnala
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I've tested the fix with PowerFlex storage migrations to see if it has any effects but all are working fine. This was initially added during development of powerflex volume migrations.
|
thanks a lot @rohityadavcloud @harikrishna-patnala for your discussion and verification ! |
Codecov Report
@@ Coverage Diff @@
## 4.18 #7945 +/- ##
============================================
- Coverage 13.06% 13.06% -0.01%
+ Complexity 9097 9096 -1
============================================
Files 2720 2720
Lines 257465 257464 -1
Branches 40146 40146
============================================
- Hits 33643 33634 -9
- Misses 219594 219603 +9
+ Partials 4228 4227 -1
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
|
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 6986 |
|
@blueorangutan test |
|
@weizhouapache a [SF] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
@blueorangutan test rocky8 kvm-rocky8 |
|
@weizhouapache a [SF] Trillian-Jenkins test job (rocky8 mgmt + kvm-rocky8) has been kicked to run smoke tests |
|
@blueorangutan test alma8 kvm-alma8 |
|
@weizhouapache a [SF] Trillian-Jenkins test job (alma8 mgmt + kvm-alma8) has been kicked to run smoke tests |
|
@blueorangutan test ubuntu22 kvm-ubuntu22 |
|
@weizhouapache a [SF] Trillian-Jenkins test job (ubuntu22 mgmt + kvm-ubuntu22) has been kicked to run smoke tests |
|
Kudos, SonarCloud Quality Gate passed!
|
|
[SF] Trillian test result (tid-7633)
|
|
[SF] Trillian test result (tid-7638)
|
|
[SF] Trillian test result (tid-7634)
|
|
[SF] Trillian test result (tid-7635)
|









Description
This PR fixes #7942
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?