Bug 1614748
| Summary: | Progress is incorrectly reported when transport method is SSH | ||
|---|---|---|---|
| Product: | [oVirt] ovirt-ansible-collection | Reporter: | Fabien Dupont <fdupont> |
| Component: | v2v-conversion-host | Assignee: | Tomáš Golembiovský <tgolembi> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Yadnyawalk Tale <ytale> |
| Severity: | high | Docs Contact: | |
| Priority: | high | ||
| Version: | 1.1.10 | CC: | bthurber, kkulkarn |
| Target Milestone: | ovirt-4.2.6 | Flags: | rule-engine:
ovirt-4.2?
tgolembi: planning_ack? rule-engine: devel_ack+ rule-engine: testing_ack+ |
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | ovirt-ansible-v2v-conversion-host-1.6.1-1.el7ev | Doc Type: | Bug Fix |
| Doc Text: |
Paths of converted disks were matched incorrectly which lead to incorrect reporting of progress. This is now fixed.
|
Story Points: | --- |
| Clone Of: | Environment: | ||
| Last Closed: | 2018-09-04 06:30:53 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | Virt | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | |||
| Bug Blocks: | 1614790 | ||
Related commits: - https://github.com/oVirt/ovirt-ansible-v2v-conversion-host/commit/299569ed4c2c2977f8f3b9e29230a0b7475809ca - https://github.com/oVirt/ovirt-ansible-v2v-conversion-host/commit/cdbba193abc690965cc3b0b6a45fec58b8df4e94 ovirt-ansible-v2v-conversion-host-1.6.1-1.el7ev Verified with CFME's v2v plugin, I can successfully migrate vm with transport method is SSH.
Following wrapper.log shows backend v2v-virt-wrapper working correctly with patch.
2018-09-03 09:03:01,440:DEBUG: Initializing disk list from [u'[NFS_Datastore_1] 67-ytale-thomas-disk-virt/67-ytale-thomas-disk-virt.vmdk'] (virt-v2v-wrapper:886)
2018-09-03 09:03:01,440:DEBUG: Internal disk list: [{'path': '[NFS_Datastore_1] 67-ytale-thomas-disk-virt/67-ytale-thomas-disk-virt.vmdk', 'progress': 0}] (virt-v2v-wrapper:894)
Verified on:
CFME - 5.9.4.7.20180829201904_ced41ae
RHV - 4.2.6.4
virt-v2v-wrapper - 1.6.3-1.el7ev
Thanks Yadnyawalk. This looks OK. |
Description of problem: When calling virt-v2v-wrapper.py with SSH as the transport method, the progress is not correctly reported. The state file reports progress on the wrong file name: {"started"=>true, "disks"=>[{"path"=>"[datastore12] test_migration/test_migration-flat", "progress"=>71.54}, {"path"=>"[datastore12] test_migration/test_migration.vmdk", "progress"=>0}], "pid"=>59814, "disk_count"=>1} The expected output is: {"started"=>true, "disks"=>[{"path"=>"[datastore12] test_migration/test_migration.vmdk", "progress"=>0}], "pid"=>59814, "disk_count"=>1} Version-Release number of selected component (if applicable): 1.6.0-1 How reproducible: Always Steps to Reproduce: Run a migration with SSH transport method.