You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by "nvazquez (via GitHub)" <gi...@apache.org> on 2025/06/12 19:42:12 UTC

[PR] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   ### Description
   
   This PR extends the VMware to KVM Migration tool by displaying the `virt-v2v` and `ovftool` versions (when available) on the UI and storing them as host details.
   
   <img width="951" alt="Screenshot 2025-06-12 at 16 38 57" src="https://github.com/user-attachments/assets/a9ac4a40-c987-430c-bc85-45657648863e" />
   
   
   ### Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [ ] Bug fix (non-breaking change which fixes an issue)
   - [x] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
   - [ ] build/CI
   - [ ] test (unit or integration test code)
   
   ### Feature/Enhancement Scale or Bug Severity
   
   #### Feature/Enhancement Scale
   
   - [ ] Major
   - [ ] Minor
   
   #### Bug Severity
   
   - [ ] BLOCKER
   - [ ] Critical
   - [ ] Major
   - [ ] Minor
   - [ ] Trivial
   
   ### Screenshots (if appropriate):
   
   ### How Has This Been Tested?
   
   Tested on KVM env with 2 Ubuntu 24.04 hosts:
   - Installed virt-v2v in one host -> Verified only virt-v2v version is listed when selecting a host for migration
   - Installed ovftool on the same host -> Verify also ovftool is displayed when selecting a host for migration
   
   #### How did you try to break this feature and the system with this change?
   
   <!-- see how your change affects other areas of the code, etc. -->
   
   <!-- Please read the [CONTRIBUTING](https://github.com/apache/cloudstack/blob/main/CONTRIBUTING.md) document -->
   


-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

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


-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

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


-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @kiranchavala a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) 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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   Great many thanks @davift, passed


-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #11019:
URL: https://github.com/apache/cloudstack/pull/11019#issuecomment-2969821621

   ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/11019?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   Attention: Patch coverage is `0%` with `24 lines` in your changes missing coverage. Please review.
   > Project coverage is 0.00%. Comparing base [(`8f2735a`)](https://app.codecov.io/gh/apache/cloudstack/commit/8f2735ab46288dc7307a4aad50ffc77dd35bfa6c?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to head [(`2a6fbbe`)](https://app.codecov.io/gh/apache/cloudstack/commit/2a6fbbe4ebe8784dbc99a07c4608d49958ee3208?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   
   | [Files with missing lines](https://app.codecov.io/gh/apache/cloudstack/pull/11019?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
   |---|---|---|
   | [...java/com/cloud/agent/manager/AgentManagerImpl.java](https://app.codecov.io/gh/apache/cloudstack/pull/11019?src=pr&el=tree&filepath=engine%2Forchestration%2Fsrc%2Fmain%2Fjava%2Fcom%2Fcloud%2Fagent%2Fmanager%2FAgentManagerImpl.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZW5naW5lL29yY2hlc3RyYXRpb24vc3JjL21haW4vamF2YS9jb20vY2xvdWQvYWdlbnQvbWFuYWdlci9BZ2VudE1hbmFnZXJJbXBsLmphdmE=) | 0.00% | [12 Missing :warning: ](https://app.codecov.io/gh/apache/cloudstack/pull/11019?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [...ervisor/kvm/resource/LibvirtComputingResource.java](https://app.codecov.io/gh/apache/cloudstack/pull/11019?src=pr&el=tree&filepath=plugins%2Fhypervisors%2Fkvm%2Fsrc%2Fmain%2Fjava%2Fcom%2Fcloud%2Fhypervisor%2Fkvm%2Fresource%2FLibvirtComputingResource.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cGx1Z2lucy9oeXBlcnZpc29ycy9rdm0vc3JjL21haW4vamF2YS9jb20vY2xvdWQvaHlwZXJ2aXNvci9rdm0vcmVzb3VyY2UvTGlidmlydENvbXB1dGluZ1Jlc291cmNlLmphdmE=) | 0.00% | [8 Missing :warning: ](https://app.codecov.io/gh/apache/cloudstack/pull/11019?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [...m/resource/wrapper/LibvirtReadyCommandWrapper.java](https://app.codecov.io/gh/apache/cloudstack/pull/11019?src=pr&el=tree&filepath=plugins%2Fhypervisors%2Fkvm%2Fsrc%2Fmain%2Fjava%2Fcom%2Fcloud%2Fhypervisor%2Fkvm%2Fresource%2Fwrapper%2FLibvirtReadyCommandWrapper.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cGx1Z2lucy9oeXBlcnZpc29ycy9rdm0vc3JjL21haW4vamF2YS9jb20vY2xvdWQvaHlwZXJ2aXNvci9rdm0vcmVzb3VyY2Uvd3JhcHBlci9MaWJ2aXJ0UmVhZHlDb21tYW5kV3JhcHBlci5qYXZh) | 0.00% | [2 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/cloudstack/pull/11019?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   
   > :exclamation:  There is a different number of reports uploaded between BASE (8f2735a) and HEAD (2a6fbbe). Click for more details.
   > 
   > <details><summary>HEAD has 1 upload less than BASE</summary>
   >
   >| Flag | BASE (8f2735a) | HEAD (2a6fbbe) |
   >|------|------|------|
   >|uitests|1|0|
   ></details>
   
   <details><summary>Additional details and impacted files</summary>
   
   
   ```diff
   @@              Coverage Diff              @@
   ##               4.20   #11019       +/-   ##
   =============================================
   - Coverage     16.15%        0   -16.16%     
   =============================================
     Files          5657        0     -5657     
     Lines        497898        0   -497898     
     Branches      60374        0    -60374     
   =============================================
   - Hits          80435        0    -80435     
   + Misses       408505        0   -408505     
   + Partials       8958        0     -8958     
   ```
   
   | [Flag](https://app.codecov.io/gh/apache/cloudstack/pull/11019/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/11019/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
   | [unittests](https://app.codecov.io/gh/apache/cloudstack/pull/11019/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `17.00% <0.00%> (-0.01%)` | :arrow_down: |
   
   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/11019?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).
   
   <details><summary> :rocket: New features to boost your workflow: </summary>
   
   - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
   - :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
   </details>


-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @nvazquez 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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

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


-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @kiranchavala a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) 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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   Good work @davift !


-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

Posted by "sureshanaparti (via GitHub)" <gi...@apache.org>.
sureshanaparti commented on code in PR #11019:
URL: https://github.com/apache/cloudstack/pull/11019#discussion_r2149800340


##########
api/src/main/java/com/cloud/host/Host.java:
##########
@@ -57,6 +57,10 @@ public static String[] toStrings(Host.Type... types) {
     public static final String HOST_VOLUME_ENCRYPTION = "host.volume.encryption";
     public static final String HOST_INSTANCE_CONVERSION = "host.instance.conversion";

Review Comment:
   @nvazquez this detail - 'host.instance.conversion' would be redundant when virt-v2v version is added in the details. so, can you remove this detail in upgrade path or when 'host.virtv2v.version' details is added?
   
   return host supports conversion when virtv2v version is there in detail.



-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   Great, many thanks @davift - Lucian's handle is @NuxRo 


-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

Posted by "sureshanaparti (via GitHub)" <gi...@apache.org>.
sureshanaparti commented on code in PR #11019:
URL: https://github.com/apache/cloudstack/pull/11019#discussion_r2149800340


##########
api/src/main/java/com/cloud/host/Host.java:
##########
@@ -57,6 +57,10 @@ public static String[] toStrings(Host.Type... types) {
     public static final String HOST_VOLUME_ENCRYPTION = "host.volume.encryption";
     public static final String HOST_INSTANCE_CONVERSION = "host.instance.conversion";

Review Comment:
   @nvazquez this details would be redundant when virt-v2v version is added in the details. so, can you remove this detail in upgrade path or when 'host.virtv2v.version' details is added?



##########
api/src/main/java/com/cloud/host/Host.java:
##########
@@ -57,6 +57,10 @@ public static String[] toStrings(Host.Type... types) {
     public static final String HOST_VOLUME_ENCRYPTION = "host.volume.encryption";
     public static final String HOST_INSTANCE_CONVERSION = "host.instance.conversion";
 
+    // Vmware to KVM Migration
+    String KVM_HOST_OVFTOOL_VERSION = "kvm.host.ovftool.version";

Review Comment:
   ```suggestion
       String KVM_HOST_OVFTOOL_VERSION = "host.ovftool.version";
   ```



-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

Posted by "nvazquez (via GitHub)" <gi...@apache.org>.
nvazquez commented on code in PR #11019:
URL: https://github.com/apache/cloudstack/pull/11019#discussion_r2149915764


##########
api/src/main/java/com/cloud/host/Host.java:
##########
@@ -57,6 +57,10 @@ public static String[] toStrings(Host.Type... types) {
     public static final String HOST_VOLUME_ENCRYPTION = "host.volume.encryption";
     public static final String HOST_INSTANCE_CONVERSION = "host.instance.conversion";

Review Comment:
   Hi @sureshanaparti I think its clearer to have those details separate - I have addressed your other comments



-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @nvazquez 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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   <b>[SF] Trillian Build Failed (tid-13560)<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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @sureshanaparti a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) 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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   <b>[SF] Trillian test result (tid-13577)</b>
   Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8
   Total time taken: 59327 seconds
   Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr11019-t13577-kvm-ol8.zip
   Smoke tests completed. 141 look OK, 0 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   


-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   > Great, many thanks @davift - Lucian's handle is @NuxRo
   
   now his cover is blown 🗡️ 


-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

Posted by "sureshanaparti (via GitHub)" <gi...@apache.org>.
sureshanaparti commented on code in PR #11019:
URL: https://github.com/apache/cloudstack/pull/11019#discussion_r2149800340


##########
api/src/main/java/com/cloud/host/Host.java:
##########
@@ -57,6 +57,10 @@ public static String[] toStrings(Host.Type... types) {
     public static final String HOST_VOLUME_ENCRYPTION = "host.volume.encryption";
     public static final String HOST_INSTANCE_CONVERSION = "host.instance.conversion";

Review Comment:
   @nvazquez this detail - 'host.instance.conversion' would be redundant when virt-v2v version is added in the details. so, can you remove this detail in upgrade path or when 'host.virtv2v.version' details is added?



-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

Posted by "sureshanaparti (via GitHub)" <gi...@apache.org>.
sureshanaparti commented on code in PR #11019:
URL: https://github.com/apache/cloudstack/pull/11019#discussion_r2149801923


##########
api/src/main/java/com/cloud/host/Host.java:
##########
@@ -57,6 +57,10 @@ public static String[] toStrings(Host.Type... types) {
     public static final String HOST_VOLUME_ENCRYPTION = "host.volume.encryption";
     public static final String HOST_INSTANCE_CONVERSION = "host.instance.conversion";
 
+    // Vmware to KVM Migration
+    String KVM_HOST_OVFTOOL_VERSION = "kvm.host.ovftool.version";
+    String KVM_HOST_VIRTV2V_VERSION = "kvm.host.virtv2v.version";

Review Comment:
   ```suggestion
       String KVM_HOST_VIRTV2V_VERSION = "host.virtv2v.version";
   ```



-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   @nvazquez 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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   <b>[SF] Trillian Build Failed (tid-13561)<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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

Posted by "nvazquez (via GitHub)" <gi...@apache.org>.
nvazquez closed pull request #11019: [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration
URL: https://github.com/apache/cloudstack/pull/11019


-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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


-- 
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] [Vmware to KVM Migration] Display virt-v2v and ovftool versions for supported hosts for migration [cloudstack]

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

   Dear @nvazquez ,
   
   During a conversation with Lucian yesterday about the `virt-v2v` “hack” he presented in his talk at the CloudStack Collaboration Conference 2025, I promised to share a containerized solution that allows running the latest `virt-v2v` on any operating system.
   
   You can find it here: [https://github.com/davift/lazy-virt-v2v](https://github.com/davift/lazy-virt-v2v)
   
   Please share this with him, as I don’t have his GitHub handle.
   
   Best regards!


-- 
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