Bug 1570407
Summary: | virt-v2v -i ova gives checksum error when missing files are mentioned in .mf | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Red Hat Enterprise Linux 7 | Reporter: | Richard W.M. Jones <rjones> | ||||||||||
Component: | libguestfs | Assignee: | Richard W.M. Jones <rjones> | ||||||||||
Status: | CLOSED ERRATA | QA Contact: | Virtualization Bugs <virt-bugs> | ||||||||||
Severity: | unspecified | Docs Contact: | |||||||||||
Priority: | unspecified | ||||||||||||
Version: | 7.5 | CC: | mxie, mzhan, nsimsolo, ptoscano, rjones, tzheng | ||||||||||
Target Milestone: | rc | ||||||||||||
Target Release: | --- | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Whiteboard: | V2V | ||||||||||||
Fixed In Version: | libguestfs-1.38.1-1.el7 | Doc Type: | If docs needed, set a value | ||||||||||
Doc Text: | Story Points: | --- | |||||||||||
Clone Of: | Environment: | ||||||||||||
Last Closed: | 2018-10-30 07:45:35 UTC | Type: | Bug | ||||||||||
Regression: | --- | Mount Type: | --- | ||||||||||
Documentation: | --- | CRM: | |||||||||||
Verified Versions: | Category: | --- | |||||||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||||
Cloudforms Team: | --- | Target Upstream Version: | |||||||||||
Embargoed: | |||||||||||||
Bug Depends On: | 1551055 | ||||||||||||
Bug Blocks: | |||||||||||||
Attachments: |
|
Description
Richard W.M. Jones
2018-04-22 16:07:15 UTC
Created attachment 1425375 [details] bz1570407-reproducer.ova To reproduce the bug, download the attachment and run: $ virt-v2v -i ova bz1570407-reproducer.ova -o null [ 0.0] Opening the source -i ova bz1570407-reproducer.ova tar: RHEL7_3_042218_extra-disk1.vmdk: Not found in archive tar: Exiting with failure status due to previous errors virt-v2v: error: checksum of disk RHEL7_3_042218_extra-disk1.vmdk does not match manifest /var/tmp/ova.2rqgzX/RHEL7_3_042218_extra.mf (actual SHA1(RHEL7_3_042218_extra-disk1.vmdk) = da39a3ee5e6b4b0d3255bfef95601890afd80709, expected SHA1(RHEL7_3_042218_extra-disk1.vmdk) = edb35db6af8384832ac27d63773ee0bb2a15e0bd) I posted some preliminary work on this: https://www.redhat.com/archives/libguestfs/2018-April/msg00163.html However this is not a full fix. It will require major work on the -i ova input mode to deal with these files (which IMHO are basically broken). Is it a real-world OVA? At least from a cursory glance it'd seem generated by VMware, is it correct? Yes it is a real world OVA (generated by VMware Version 6.0.0 Build 2594327). I believe it has something to do with snapshots. The file list is: RHEL7_3_042218_extra.ovf RHEL7_3_042218_extra.mf RHEL7_3_042218_extra-disk1.vmdk.000000000 I will attach the OVF and MF files. Created attachment 1425858 [details]
RHEL7_3_042218_extra.ovf
Created attachment 1425859 [details]
RHEL7_3_042218_extra.mf
You can see the original problem from the bug report because the .mf file references a non-existent file: SHA1(RHEL7_3_042218_extra-disk1.vmdk)= ca3727b9c701ffe2552e4809b3ecf9d85a82873c My patch series solve this problem. However there's a second problem in the .ovf file which also references a non-existent file: <References> <File ovf:chunkSize="7516192768" ovf:href="RHEL7_3_042218_extra-disk1.vmdk" ovf:id="file1" ovf:size="721755648"/> </References> and fixing this involves some deeper surgery on v2v/input_ova.ml (which to be fair we probably need to do at some time). This should be a full fix: https://www.redhat.com/archives/libguestfs/2018-April/msg00167.html Using this I was able to convert the test image which Nisim supplied to me privately. Upstream in: e5946165c v2v: -i ova: Handle OVAs containing snapshots (RHBZ#1570407). 80d6d9396 v2v: -i ova: Sanity check hrefs to ensure they stay inside the tarball. e24771468 v2v: -i ova: Replace subdirectory function with clearer inline code. 075333ede common: mlstdutils: Inline unsafe ‘subdirectory’ function. 184dba598 v2v: -i ova: Factor out code for dealing with OVA files. ece2c60c5 v2v: parse OVF: Export useful parse_disks function. d1e95a30f v2v: Ignore miscellaneous tar messages when parsing tar for file locations. 112e3bc31 v2v: -i ova: Ignore non-existent files mentioned in *.mf. 190333b8b mltools: Checksums.verify_checksum{,s} returns an error type instead of throwing exception. Fixes backported in 1.38.1. I can reproduce the bug using attached ova with below builds: virt-v2v-1.38.0-1.el7.x86_64 libguestfs-1.38.0-1.el7.x86_64 Reproduce steps: 1.# virt-v2v -i ova bz1570407-reproducer.ova -o null [ 0.0] Opening the source -i ova bz1570407-reproducer.ova virt-v2v: warning: making OVA directory public readable to work around libvirt bug https://bugzilla.redhat.com/1045069 tar: RHEL7_3_042218_extra-disk1.vmdk: Not found in archive tar: Exiting with failure status due to previous errors virt-v2v: error: checksum of disk RHEL7_3_042218_extra-disk1.vmdk does not match manifest /var/tmp/ova.jZTuqW/RHEL7_3_042218_extra.mf (actual SHA1(RHEL7_3_042218_extra-disk1.vmdk) = da39a3ee5e6b4b0d3255bfef95601890afd80709, expected SHA1(RHEL7_3_042218_extra-disk1.vmdk) = edb35db6af8384832ac27d63773ee0bb2a15e0bd) If reporting bugs, run virt-v2v with debugging enabled and include the complete output: virt-v2v -v -x [...] By the way, I can't export a ova like bug's attached one even if vmware 6.0 guest has snapshot so far. Verify the bug with builds: virt-v2v-1.38.1-1.el7.x86_64 libguestfs-1.38.1-1.el7.x86_64 Steps: 1.# virt-v2v -i ova bz1570407-reproducer.ova -o null [ 0.0] Opening the source -i ova bz1570407-reproducer.ova virt-v2v: warning: making OVA directory public readable to work around libvirt bug https://bugzilla.redhat.com/1045069 virt-v2v: warning: manifest has a checksum for non-existent file RHEL7_3_042218_extra-disk1.vmdk (ignored) [ 0.4] Creating an overlay to protect the source from being modified qemu-img: /var/tmp/v2vovlb05920.qcow2: Failed to read footer: Input/output error Could not open backing image to determine size. virt-v2v: error: qemu-img command failed, see earlier errors If reporting bugs, run virt-v2v with debugging enabled and include the complete output: virt-v2v -v -x [...] Hi rjones. I can see latest virt-v2v has improved the warning for non-existent file in mf,but v2v conversion is failed in my test (according to comment10, you could convert attached ova), could you please help to check? Created attachment 1434246 [details]
v2v-1.38.1-1.log
> Verify the bug with builds:
> virt-v2v-1.38.1-1.el7.x86_64
> libguestfs-1.38.1-1.el7.x86_64
>
> Steps:
> 1.# virt-v2v -i ova bz1570407-reproducer.ova -o null
> [ 0.0] Opening the source -i ova bz1570407-reproducer.ova
> virt-v2v: warning: making OVA directory public readable to work around
> libvirt bug https://bugzilla.redhat.com/1045069
> virt-v2v: warning: manifest has a checksum for non-existent file
> RHEL7_3_042218_extra-disk1.vmdk (ignored)
> [ 0.4] Creating an overlay to protect the source from being modified
> qemu-img: /var/tmp/v2vovlb05920.qcow2: Failed to read footer: Input/output
> error
> Could not open backing image to determine size.
> virt-v2v: error: qemu-img command failed, see earlier errors
Can you please try as non-root user, or to move the .ova file where even non-root users can access it (e.g. /tmp, /var/tmp, etc)?
Hi Pino, The conversion is still failed after moving ova file to /var/tmp and running command as non-root user [mxie@hp-dl380eg8-02 home]$ virt-v2v -i ova bz1570407-reproducer.ova -o null [ 0.0] Opening the source -i ova bz1570407-reproducer.ova virt-v2v: warning: manifest has a checksum for non-existent file RHEL7_3_042218_extra-disk1.vmdk (ignored) [ 0.4] Creating an overlay to protect the source from being modified qemu-img: /var/tmp/v2vovl528738.qcow2: Failed to read footer: Input/output error Could not open backing image to determine size. virt-v2v: error: qemu-img command failed, see earlier errors If reporting bugs, run virt-v2v with debugging enabled and include the complete output: virt-v2v -v -x [...] (In reply to mxie from comment #17) > Hi Pino, > > The conversion is still failed after moving ova file to /var/tmp and > running command as non-root user > > [mxie@hp-dl380eg8-02 home]$ virt-v2v -i ova bz1570407-reproducer.ova -o null > [ 0.0] Opening the source -i ova bz1570407-reproducer.ova > virt-v2v: warning: manifest has a checksum for non-existent file > RHEL7_3_042218_extra-disk1.vmdk (ignored) So please note here that bz1570407-reproducer.ova is not a complete, working OVA. It's basically something I just hacked up to reproduce the bug. If virt-v2v has been patched to fix the bug then trying to actually convert bz1570407-reproducer.ova will fail. I will send you a link to the actual OVA which caused the original bug report (but we cannot redistribute it). Use the ova file provided by rjones to test the bug again Reproduce the bug with builds virt-v2v-1.38.0-1.el7.x86_64 libguestfs-1.38.0-1.el7.x86_64 Reproduce steps: 1.Prepare a ova file and there is missing file mentioned in .mf # tar tf RHEL7_3_042218_extra.ova RHEL7_3_042218_extra.ovf RHEL7_3_042218_extra.mf RHEL7_3_042218_extra-disk1.vmdk.000000000 # cat RHEL7_3_042218_extra.mf SHA1(RHEL7_3_042218_extra.ovf)= bf626b725f3bd251cc37771dd6ce48c9915f00c3 SHA1(RHEL7_3_042218_extra-disk1.vmdk.000000000)= ca3727b9c701ffe2552e4809b3ecf9d85a82873c SHA1(RHEL7_3_042218_extra-disk1.vmdk)= ca3727b9c701ffe2552e4809b3ecf9d85a82873c 2.Use virt-v2v to convert this ova file # virt-v2v -i ova RHEL7_3_042218_extra.ova -o null [ 0.0] Opening the source -i ova RHEL7_3_042218_extra.ova tar: RHEL7_3_042218_extra-disk1.vmdk: Not found in archive tar: Exiting with failure status due to previous errors virt-v2v: error: checksum of disk RHEL7_3_042218_extra-disk1.vmdk does not match manifest /var/tmp/ova.T82C33/RHEL7_3_042218_extra.mf (actual SHA1(RHEL7_3_042218_extra-disk1.vmdk) = da39a3ee5e6b4b0d3255bfef95601890afd80709, expected SHA1(RHEL7_3_042218_extra-disk1.vmdk) = ca3727b9c701ffe2552e4809b3ecf9d85a82873c) If reporting bugs, run virt-v2v with debugging enabled and include the complete output: virt-v2v -v -x [...] Verify the bug with builds: virt-v2v-1.38.2-1.el7.x86_64 libguestfs-1.38.2-1.el7.x86_64 Steps: 1.Convert the ova file which has missing file mentioned in .mf by virt-v2v # virt-v2v -i ova RHEL7_3_042218_extra.ova -o null [ 0.0] Opening the source -i ova RHEL7_3_042218_extra.ova virt-v2v: warning: manifest has a checksum for non-existent file RHEL7_3_042218_extra-disk1.vmdk (ignored) [ 4.3] Creating an overlay to protect the source from being modified [ 4.7] Initializing the target -o null [ 4.7] Opening the overlay [ 74.3] Inspecting the overlay [ 82.1] Checking for sufficient free disk space in the guest [ 82.1] Estimating space required on target for each disk [ 82.1] Converting Red Hat Enterprise Linux Client 7.0 (Maipo) to run on KVM virt-v2v: This guest has virtio drivers installed. [ 149.4] Mapping filesystem data to avoid copying unused and blank areas [ 150.5] Closing the overlay [ 151.3] Checking if the guest needs BIOS or UEFI to boot [ 151.3] Assigning disks to buses [ 151.3] Copying disk 1/1 to qemu URI json:{ "file.driver": "null-co", "file.size": "1E" } (raw) (100.00/100%) [ 171.4] Creating output metadata [ 171.4] Finishing off Result: virt-v2v warning shows correct during conversion and conversion could be finished successfully Verify the bug with builds: virt-v2v-1.38.2-3.el7.x86_64 libguestfs-1.38.2-3.el7.x86_64 libvirt-4.3.0-1.el7.x86_64 qemu-kvm-rhev-2.12.0-3.el7.x86_64 Steps: 1.Prepare a ova file which has missing file mentioned in .mf 1.1 # tar -tf RHEL7_3_042218_extra.ova RHEL7_3_042218_extra.ovf RHEL7_3_042218_extra.mf RHEL7_3_042218_extra-disk1.vmdk.000000000 1.2 # cat RHEL7_3_042218_extra.mf SHA1(RHEL7_3_042218_extra.ovf)= bf626b725f3bd251cc37771dd6ce48c9915f00c3 SHA1(RHEL7_3_042218_extra-disk1.vmdk.000000000)= ca3727b9c701ffe2552e4809b3ecf9d85a82873c SHA1(RHEL7_3_042218_extra-disk1.vmdk)= ca3727b9c701ffe2552e4809b3ecf9d85a82873c 2.Convert the ova by virt-v2v # virt-v2v -i ova RHEL7_3_042218_extra.ova -o rhv -os 10.66.144.40:/home/nfs_export -of qcow2 [ 0.0] Opening the source -i ova RHEL7_3_042218_extra.ova virt-v2v: warning: manifest has a checksum for non-existent file RHEL7_3_042218_extra-disk1.vmdk (ignored) [ 10.9] Creating an overlay to protect the source from being modified [ 12.0] Initializing the target -o rhv -os 10.66.144.40:/home/nfs_export [ 12.7] Opening the overlay [ 23.4] Inspecting the overlay [ 36.2] Checking for sufficient free disk space in the guest [ 36.2] Estimating space required on target for each disk [ 36.2] Converting Red Hat Enterprise Linux Client 7.0 (Maipo) to run on KVM virt-v2v: This guest has virtio drivers installed. [ 121.3] Mapping filesystem data to avoid copying unused and blank areas [ 123.5] Closing the overlay [ 124.5] Checking if the guest needs BIOS or UEFI to boot [ 124.5] Assigning disks to buses [ 124.5] Copying disk 1/1 to /tmp/v2v.jt81DN/ea9cb06f-8bf9-4fc8-a247-478e754d898a/images/ee8c82f5-1d05-4785-9d93-1aa18a665de7/43db4974-aa9f-4b54-b787-dcd0a867efc3 (qcow2) (100.00/100%) [ 174.5] Creating output metadata [ 174.6] Finishing off 3.Import guest from export domain to data domain, guest could be power on and boot into os normally Result: Warning shows correct during v2v conversion and conversion could be finished successfully, so move the bug from ON_QA to VERIFIED Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHEA-2018:3021 |