Bug 1056640 - virt-v2v OVA/OVF import fails when no (optional) ovf:capacityAllocationUnits is specified
Summary: virt-v2v OVA/OVF import fails when no (optional) ovf:capacityAllocationUnits ...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Virtualization Tools
Classification: Community
Component: libguestfs
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Richard W.M. Jones
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-01-22 15:43 UTC by Ronald van Zantvoort
Modified: 2015-08-27 14:33 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-04-15 15:47:57 UTC
Embargoed:


Attachments (Terms of Use)
Proposed patch to solve a few disk sizing issues (2.24 KB, patch)
2014-01-22 16:07 UTC, Ronald van Zantvoort
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1056045 0 unspecified CLOSED virt-v2v OVA import fails when no (optional) manifest file is included 2021-02-22 00:41:40 UTC
Red Hat Bugzilla 1056126 0 unspecified CLOSED virt-v2v OVA/OVF import fails with 'MegaBytes' as memory allocation units 2021-02-22 00:41:40 UTC
Red Hat Bugzilla 1056534 0 unspecified CLOSED virt-v2v OVF/OVA import fails when disks aren't attached to either IDE or SCSI controllers 2021-02-22 00:41:40 UTC
Red Hat Bugzilla 1056983 0 unspecified CLOSED virt-v2v OVA/OVF import fails when no (optional) ovf:populatedSize is specified 2021-02-22 00:41:40 UTC
Red Hat Bugzilla 1057006 0 unspecified CLOSED virt-v2v OVA/OVF import fails when no (optional) Name under VirtualSystem is specified 2021-02-22 00:41:40 UTC


Description Ronald van Zantvoort 2014-01-22 15:43:00 UTC
Description of problem:
When an OVF file under it's DiskSection defines any disk without the ovf:capacityAllocationUnits, import will fail with
Unexpected ovf:capacityAllocationUnits:  at /usr/share/perl5/vendor_perl/Sys/VirtConvert/Connection/VMwareOVASource.pm line 345.

However, according to spec, the default is bytes and the property is optional.

Version-Release number of selected component (if applicable):
Fedora 20 / virt-v2v 0.9.0

How reproducible:
Import an OVA/OVF with a DiskSection and a Disk in it without the ovf:capacityAllocationUnits but with the ovf:capacity property

Actual results:
Import fails.

Expected results:
Import succeeds with ovf:capacity interpreted as bytes 

Additional info:
The OVF standard (http://www.dmtf.org/sites/default/files/standards/documents/DSP0243_1.0.0.pdf) defines:

"The capacity of a virtual disk shall be specified by the ovf:capacity attribute with an xs:long integer 
value. The default unit of allocation shall be bytes. The optional string attribute 
ovf:capacityAllocationUnits may be used to specify a particular unit of allocation"

Comment 1 Ronald van Zantvoort 2014-01-22 16:07:51 UTC
Created attachment 853937 [details]
Proposed patch to solve a few disk sizing issues

Proposed patch for resolution of issue.

* ovf:capacityAllocationUnits becomes optional
* ovf:capacityAllocationUnits is now interpreted the same way as the patch for https://bugzilla.redhat.com/show_bug.cgi?id=1056126 proposes, with OVF-compliant parsing of the value, collapsing it into a bytes value.

Comment 2 Richard W.M. Jones 2015-04-15 15:47:57 UTC
Hi, thanks for the bug report.  Unfortunately it is filed against a
very old version of virt-v2v (0.9) which we don't support at all.

Luckily there is a newer version which almost certainly fixes your
bug already.  It's available starting in Fedora 21.  If you still
experience this bug, please try with the new version, and reopen
if it still happens.

Comment 3 Ronald van Zantvoort 2015-08-27 14:33:06 UTC
(In reply to Richard W.M. Jones from comment #1)
> Unfortunately it is filed against a
> very old version of virt-v2v (0.9) 

Well, that tends to happen if you wait 16 months with a reply to a ready2commit submitted patch.


Note You need to log in before you can comment on or make changes to this bug.