Bug 967233 - RFE: Add a migration flag to auto-create storage stubs on the destination
Summary: RFE: Add a migration flag to auto-create storage stubs on the destination
Alias: None
Product: Virtualization Tools
Classification: Community
Component: libvirt
Version: unspecified
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
Assignee: Michal Privoznik
QA Contact:
: 735073 (view as bug list)
Depends On:
TreeView+ depends on / blocked
Reported: 2013-05-25 20:49 UTC by Cole Robinson
Modified: 2014-12-09 16:05 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2014-12-09 16:05:31 UTC

Attachments (Terms of Use)

Description Cole Robinson 2013-05-25 20:49:36 UTC
Doing 'virsh migrate --copy-storage-all' currently isn't very end user friendly because it requires initial stub storage files to exist on the destination host. 

Trying to automatically do this is likely difficult and error prone given all the possible storage configurations available, but IMO as is the feature is pretty uninteresting to end users.

I think we should add a migration API flag that will have the destination libvirt create storage stubs for disk images, if required. Even if a first attempt just handles image files that would go a long way.

Without this, we would _have_ to implement something similar just to expose this in virt-manager, so IMO it's better to do this at the libvirt level first, so all clients (particularly virsh) can benefit.

Comment 1 Cole Robinson 2013-05-25 20:49:59 UTC
*** Bug 735073 has been marked as a duplicate of this bug. ***

Comment 2 Michal Privoznik 2013-05-27 04:58:56 UTC
There's a RHEL bug requsting the same 927252.

Comment 3 Michal Privoznik 2014-12-09 16:01:53 UTC
So, the patches are upstream (see https://bugzilla.redhat.com/show_bug.cgi?id=927252#c9 ). There's no special flag added. The storage will be precreated automatically, if user hasn't done it. Cole is that enough to close this bug?

Comment 4 Cole Robinson 2014-12-09 16:05:31 UTC
Yeah I saw those patches go by, thanks a lot! I still need to test it out but regardless this bug is solved, if I hit any issues ill open separate reports

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