JoaoJandre opened a new pull request, #8909:
URL: https://github.com/apache/cloudstack/pull/8909

   ### Description
   
   Using KVM as a hypervisor, when performing a live migration of a VM with a 
volume, if the volume is on local storage and the migration destination is an 
NFS storage, an error occurs during the migration. This happens because there 
is a validation to check whether the destination host has direct access to the 
source storage of the volume being migrated. However, this validation is not 
necessary, since the host that will perform the migration is the source host 
itself. Therefore, the destination host only needs access to the destination 
storage.
   
   ### Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [X] Bug fix (non-breaking change which fixes an issue)
   - [ ] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
   - [ ] build/CI
   
   ### Feature/Enhancement Scale or Bug Severity
   
   #### Feature/Enhancement Scale
   
   - [ ] Major
   - [X] Minor
   
   #### Bug Severity
   
   - [ ] BLOCKER
   - [ ] Critical
   - [ ] Major
   - [X] Minor
   - [ ] Trivial
   
   
   ### Screenshots (if appropriate):
   
   
   ### How Has This Been Tested?
   
   Before this PR, the following live volume migration cases failed in KVM:
   
   | Volume |  Origin |  Destination |
   | ------ | ------ | ------ |
   | ROOT | Local | NFS1 |
   | DATA1 | Local | NFS1 |
   
   | Volume |  Origin |  Destination |
   | ------ | ------ | ------ |
   | ROOT | Local | NFS1 |
   
   With this PR, the above cases now work. Furthermore, the following cases 
where validated, to make sure that no regressions were introduced:
   
   | Volume |  Origin |  Destination |
   | ------ | ------ | ------ |
   | ROOT | NFS1 | NFS2 |
   | DATA1 | NFS1 | NFS2 |
   
   | Volume |  Origin |  Destination |
   | ------ | ------ | ------ |
   | ROOT | NFS1 | Local |
   | DATA1 | NFS1 | Local |
   
   | Volume |  Origin |  Destination |
   | ------ | ------ | ------ |
   | ROOT | NFS1 | ISCSI1 |
   | DATA1 | NFS1 | ISCSI1 |
   
   | Volume |  Origin |  Destination |
   | ------ | ------ | ------ |
   | ROOT | ISCSI1 | NFS1 |
   | DATA1 | ISCSI1 | NFS1 |
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to