Bug 1469077 - [RFE] Import from kvm source when guest's disk type is volume
Summary: [RFE] Import from kvm source when guest's disk type is volume
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: vdsm
Classification: oVirt
Component: RFEs
Version: 4.19.20
Hardware: x86_64
OS: Unspecified
medium
medium
Target Milestone: ovirt-4.2.0
: ---
Assignee: Tomáš Golembiovský
QA Contact: Nisim Simsolo
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-07-10 11:01 UTC by Tomáš Golembiovský
Modified: 2019-04-28 14:08 UTC (History)
16 users (show)

Fixed In Version:
Clone Of: 1468509
Environment:
Last Closed: 2017-12-20 10:51:32 UTC
oVirt Team: Virt
Embargoed:
michal.skrivanek: ovirt-4.2?
gklein: testing_plan_complete+
rule-engine: planning_ack?
michal.skrivanek: devel_ack+
rule-engine: testing_ack+


Attachments (Terms of Use)
vdsm.log (starts at 2017-10-02 15:59:42) (447.41 KB, application/x-xz)
2017-10-02 13:05 UTC, Nisim Simsolo
no flags Details
engine.log (488.31 KB, application/x-xz)
2017-10-02 13:06 UTC, Nisim Simsolo
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 79834 0 master MERGED v2v: allow import of disk with type='volume' 2020-02-04 08:57:45 UTC

Comment 1 Tomáš Golembiovský 2017-07-10 11:07:32 UTC
*** Bug 1468509 has been marked as a duplicate of this bug. ***

Comment 2 Michal Skrivanek 2017-07-11 10:26:12 UTC
*** Bug 1468944 has been marked as a duplicate of this bug. ***

Comment 3 Tomáš Golembiovský 2017-07-11 13:24:34 UTC
For the context: type='volume' allows user to use disks from storage pool without caring about details how the pool was defined.

For example with the pool 'pool1' deined like:

    <pool type="dir">
      <name>pool1</name>
      <target>
        <path>/var/vm_pool/</path>
      </target>
    </pool>

The following disk definitions are equivalent:

    <disk type='file' device='disk'> 
      <source file='/var/vm_pool/machine1.qcow2'/>
      ...
    </disk>

and

    <disk type='volume' device='disk'> 
      <source pool='pool1' volume='machine1.qcow2'/>
      ...
    </disk>

Comment 7 Nisim Simsolo 2017-10-02 13:05:08 UTC
Created attachment 1333230 [details]
vdsm.log (starts at 2017-10-02 15:59:42)

Comment 8 Nisim Simsolo 2017-10-02 13:06:08 UTC
Created attachment 1333231 [details]
engine.log

Comment 9 Tomáš Golembiovský 2017-10-19 12:13:02 UTC
The disk definition is wrong:

    <disk type='volume' device='disk'>
      <driver name='qemu' type='raw'/>
      <target dev='hda' bus='ide'/>
      <address type='drive' controller='0' bus='0' target='0' unit='0'/>
    </disk>

There is no <source> element here. You need to add something like:

    <source pool='some_pool' volume='some_volume.qcow2' />

Comment 10 Emma Heftman 2017-10-24 10:57:08 UTC
Hi Tomas
I'm reviewing the doc text and it is not clear to me what you mean by "In this case the disk is described as a storage pool/volume pair."

Also, is there any impact on documentation? i.e. is there something that must be manually configure in order for this to work?
Thanks
Emma

Comment 11 Tomáš Golembiovský 2017-10-24 14:32:59 UTC
(In reply to Emma Heftman from comment #10)
> Hi Tomas
> I'm reviewing the doc text and it is not clear to me what you mean by "In
> this case the disk is described as a storage pool/volume pair."

This tells how the disk is defined in libvirt XML of the VM on the source KVM machine.


> Also, is there any impact on documentation? i.e. is there something that
> must be manually configure in order for this to work?

No.

Comment 12 Emma Heftman 2017-10-25 12:21:37 UTC
(In reply to Tomáš Golembiovský from comment #11)
> (In reply to Emma Heftman from comment #10)
> > Hi Tomas
> > I'm reviewing the doc text and it is not clear to me what you mean by "In
> > this case the disk is described as a storage pool/volume pair."
> 
> This tells how the disk is defined in libvirt XML of the VM on the source
> KVM machine.
> 
Thanks Tomas. I believe this is too detailed for the Release Notes which describes what the fix/enhancement does, and not how that was achieved.
 
> > Also, is there any impact on documentation? i.e. is there something that
> > must be manually configure in order for this to work?
> 
> No.

Comment 13 Nisim Simsolo 2017-11-01 09:27:33 UTC
Verification builds:
ovirt-engine-4.2.0-0.0.master.20171029154613.git19686f3.el7.centos
vdsm-4.20.4-12.git11d6d3d.el7.centos.x86_64
libvirt-client-3.2.0-14.el7_4.3.x86_64
qemu-kvm-common-ev-2.9.0-16.el7_4.8.1.x86_64

Polarion verification test case added to external trackers.

Comment 14 Sandro Bonazzola 2017-12-20 10:51:32 UTC
This bugzilla is included in oVirt 4.2.0 release, published on Dec 20th 2017.

Since the problem described in this bug report should be
resolved in oVirt 4.2.0 release, published on Dec 20th 2017, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.


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