You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by "JoaoJandre (via GitHub)" <gi...@apache.org> on 2024/04/12 18:52:38 UTC

[PR] Fix migration from local storage to NFS in KVM [cloudstack]

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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2052350097

   ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8909?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   All modified and coverable lines are covered by tests :white_check_mark:
   > Project coverage is 4.31%. Comparing base [(`a5508ac`)](https://app.codecov.io/gh/apache/cloudstack/commit/a5508acc54d1638ba469f4d7ce9898b81b5209d0?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to head [(`0bec7da`)](https://app.codecov.io/gh/apache/cloudstack/pull/8909?dropdown=coverage&src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   > Report is 561 commits behind head on main.
   
   
   <details><summary>Additional details and impacted files</summary>
   
   
   ```diff
   @@             Coverage Diff              @@
   ##               main   #8909       +/-   ##
   ============================================
   - Coverage     13.17%   4.31%    -8.86%     
   ============================================
     Files          2725     362     -2363     
     Lines        258235   29176   -229059     
     Branches      40249    5120    -35129     
   ============================================
   - Hits          34013    1260    -32753     
   + Misses       219913   27774   -192139     
   + Partials       4309     142     -4167     
   ```
   
   | [Flag](https://app.codecov.io/gh/apache/cloudstack/pull/8909/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
   |---|---|---|
   | [uitests](https://app.codecov.io/gh/apache/cloudstack/pull/8909/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `4.31% <ø> (?)` | |
   
   Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
   
   
   </details>
   
   [:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/cloudstack/pull/8909?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).   
   :loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2060628660

   @DaanHoogland a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "sureshanaparti (via GitHub)" <gi...@apache.org>.
sureshanaparti commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2055700136

   @blueorangutan package


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "JoaoJandre (via GitHub)" <gi...@apache.org>.
JoaoJandre commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2056627084

   @blueorangutan package


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "rohityadavcloud (via GitHub)" <gi...@apache.org>.
rohityadavcloud merged PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "JoaoJandre (via GitHub)" <gi...@apache.org>.
JoaoJandre commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2165430245

   > Thanks @JoaoJandre, we have the option when changing disk offering to migrate to another storage if required, why not fix it there instead? ![Screenshot 2024-06-12 at 11 44 52](https://private-user-images.githubusercontent.com/13551960/338891518-e56ec583-a57a-4941-9cb0-70f64450b2f4.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTgyMDI0MDgsIm5iZiI6MTcxODIwMjEwOCwicGF0aCI6Ii8xMzU1MTk2MC8zMzg4OTE1MTgtZTU2ZWM1ODMtYTU3YS00OTQxLTljYjAtNzBmNjQ0NTBiMmY0LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA2MTIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNjEyVDE0MjE0OFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWZiNWRlODMyZjEzYzVhOTQ4Y2FhYjUwOTU2ZGRjYWJjYjA5YmQxN2NhMzEwOTFiNjU0NTk4MmEzYjczNDVkYjkmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.bYYtTWcCt02IVLLWwoRaFLjgYwm71BLnyTDZG-xTL
 IE)
   > 
   > I think it's more lean approach to execute this operation since in the case you've described you might end up with volume on shared storage while having assigned offering of local storage and vice versa. Currently migrating cross different types of storage seems not working ( returns 'Index 0 out of bounds for length 0' ), would you consider moving the fix there?
   
   
   Hey @borisstoyanov, 
   
   I think you misunderstood the PR. The issue is not with the disk offering, but with a validation that makes no sense: the destination host does not need to have access to the original storage, regardless of the disk offering. This PR fixes the current validation to reflect that. 
   
   About ending up with a disk offering in a storage with different type, this is current behavior, as introduced in #2425. It is assumed that when the ROOT admin migrates the volume somewhere, they know what they are doing. Same applies to migrating VMs to hosts that do not have a certain tag, the root admin has this power.
   
   I didn't understand your last point, are you saying that this PR broke migration across different types of storage, or that this is a current issue? if so, could you give me the link to the issue?


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "borisstoyanov (via GitHub)" <gi...@apache.org>.
borisstoyanov commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2160545695

   @JoaoJandre thanks for this fix, I'm trying to reproduce the issue and bug fix, what APIs are you using the replicate this situation. I'm questioning how do you migrate a volume with shared storage offering to local storage? 


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2115819271

   @JoaoJandre a [SL] 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.


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "DaanHoogland (via GitHub)" <gi...@apache.org>.
DaanHoogland commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2124154720

   @blueorangutan test


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2125799576

   <b>[SF] Trillian test result (tid-10240)</b>
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 46429 seconds
   Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8909-t10240-kvm-centos7.zip
   Smoke tests completed. 130 look OK, 1 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_events_resource | `Error` | 416.26 | test_events_resource.py
   


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "DaanHoogland (via GitHub)" <gi...@apache.org>.
DaanHoogland commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2056101062

   @JoaoJandre : 
   ```
   09:11:09 [ERROR] /jenkins/workspace/acs-centos7-pkg-builder/dist/rpmbuild/BUILD/cloudstack-4.19.1.0-SNAPSHOT/engine/storage/datamotion/src/test/java/org/apache/cloudstack/storage/motion/StorageSystemDataMotionStrategyTest.java:26:15: Unused import - org.mockito.Mockito.never. [UnusedImports]
   09:11:09 [ERROR] /jenkins/workspace/acs-centos7-pkg-builder/dist/rpmbuild/BUILD/cloudstack-4.19.1.0-SNAPSHOT/engine/storage/datamotion/src/test/java/org/apache/cloudstack/storage/motion/StorageSystemDataMotionStrategyTest.java:51:8: Unused import - org.mockito.verification.VerificationMode. [UnusedImports]
   09:11:09 [ERROR] /jenkins/workspace/acs-centos7-pkg-builder/dist/rpmbuild/BUILD/cloudstack-4.19.1.0-SNAPSHOT/engine/storage/datamotion/src/test/java/org/apache/cloudstack/storage/motion/StorageSystemDataMotionStrategyTest.java:65:8: Unused import - java.util.LinkedList. [UnusedImports]
   ```


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2056629655

   @JoaoJandre a [SL] 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.


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2124155693

   @DaanHoogland a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2056789516

   Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 9261


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "DaanHoogland (via GitHub)" <gi...@apache.org>.
DaanHoogland commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2060626189

   @blueorangutan test


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "JoaoJandre (via GitHub)" <gi...@apache.org>.
JoaoJandre commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2160728792

   > @JoaoJandre thanks for this fix, I'm trying to reproduce the issue and bug fix, what APIs are you using the replicate this situation. I'm questioning how do you migrate a volume with shared storage offering to local storage?
   
   I did all tests using live VM + volume migration on the UI. I was able to migrate volumes from NFS to local and from local to NFS this way. Looking at the API call that the UI made, it was using `migrateVirtualMachineWithVolume`. Something like:
   ```
   http://<mgmtserver>/client/api/?hostid=e8050fa0-c834-47c5-9a1e-089595aa2edc&virtualmachineid=27640463-d916-4e9b-b223-4f3c84fc828a&migrateto[0].volume=31df1fe1-53f2-4ff5-9b86-d75f6f2afe85&migrateto[0].pool=49a3f50d-e70c-4963-8e6a-f70e3a9d295d&command=migrateVirtualMachineWithVolume&response=json
   ```


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2062722778

   <b>[SF] Trillian test result (tid-9875)</b>
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 46521 seconds
   Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8909-t9875-kvm-centos7.zip
   Smoke tests completed. 127 look OK, 2 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_events_resource | `Error` | 296.90 | test_events_resource.py
   test_01_events_resource | `Error` | 296.91 | test_events_resource.py
   test_04_deploy_vm_for_other_user_and_test_vm_operations | `Failure` | 84.97 | test_network_permissions.py
   ContextSuite context=TestNetworkPermissions>:teardown | `Error` | 1.43 | test_network_permissions.py
   


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "JoaoJandre (via GitHub)" <gi...@apache.org>.
JoaoJandre commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2052424087

   > @JoaoJandre is this also applicable fix for 4.19?
   
   @rohityadavcloud yes, I'll rebase with 4.19 and target it


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "JoaoJandre (via GitHub)" <gi...@apache.org>.
JoaoJandre commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2166499037

   > I apologise @JoaoJandre, I thought this was much deeper change, what you suggest is making sense now, and the issues I'm pointing out are not related to your PR. LGTM
   
   No problem @borisstoyanov :)


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2115938802

   Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 9622


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "JoaoJandre (via GitHub)" <gi...@apache.org>.
JoaoJandre commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2115817219

   @blueorangutan package


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2055706378

   @sureshanaparti a [SL] 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.


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2060686088

   <b>[SF] Trillian Build Failed (tid-9871)<b/>


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "rohityadavcloud (via GitHub)" <gi...@apache.org>.
rohityadavcloud commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2052416091

   @JoaoJandre is this also applicable fix for 4.19?


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2055776164

   Packaging result [SF]: ✖️ el7 ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 9251


-- 
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-unsubscribe@cloudstack.apache.org

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


Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

Posted by "JoaoJandre (via GitHub)" <gi...@apache.org>.
JoaoJandre commented on PR #8909:
URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2059577845

   @sureshanaparti could we run the CI here?


-- 
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-unsubscribe@cloudstack.apache.org

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