Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration

2017-01-14 Thread Jasmin J.
Hi! > I'll try work on this next week. May I ask you to consider a new API for merging between Storage and Plugin. It might be very useful for a plugin to know a migration is starting and is finished, so that a plugin can do additional commands to the particular storage driver. And a plugin needs

Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration

2017-01-13 Thread Alexandre DERUMIER
Wolfgang Bumiller" À: "aderumier" Cc: "dietmar" , "pve-devel" Envoyé: Vendredi 13 Janvier 2017 11:56:41 Objet: Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration On Tue, Jan 10, 2017 at 08:19:29AM +0100, Alexandre DERUMIER wrote: &g

Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration

2017-01-13 Thread Wolfgang Bumiller
On Tue, Jan 10, 2017 at 08:19:29AM +0100, Alexandre DERUMIER wrote: > Thanks Wolfgang, > > > >>Basically storage plugins should have to define > >>- a set of formats they can export as and import from > >>- whether these formats can include snapshots > >>- a priority > > that's exactly what

Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration

2017-01-09 Thread Alexandre DERUMIER
e bit for your patches . Thanks for the hard work on this ! - Mail original - De: "Wolfgang Bumiller" À: "aderumier" Cc: "dietmar" , "pve-devel" Envoyé: Lundi 9 Janvier 2017 12:36:12 Objet: Re: [pve-devel] [PATCH] add with-local-disks option for l

Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration

2017-01-09 Thread Wolfgang Bumiller
On Sat, Jan 07, 2017 at 03:16:22PM +0100, Alexandre DERUMIER wrote: > >>I think wolfgang has some experimental code to implement > >>kind of send/receive for most storage types .. I guess this > >>could be useful here. > > maybe it could be great to have something in pve-storage plugins. > > f

Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration

2017-01-07 Thread Alexandre DERUMIER
inal - De: "dietmar" À: "aderumier" Cc: "pve-devel" Envoyé: Vendredi 6 Janvier 2017 16:49:33 Objet: Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration > >>No. But my question was more like: why not? > >>I guess this is not

Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration

2017-01-06 Thread Dietmar Maurer
> >>No. But my question was more like: why not? > >>I guess this is not really an important feature, because users > >>can migrate single disk to another storage anyways... > > I'll try to look at this. > > it could be usefull too for full cloning a template with local disk, to a > remote node

Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration

2017-01-06 Thread Alexandre DERUMIER
emote node. - Mail original - De: "dietmar" À: "aderumier" Cc: "pve-devel" Envoyé: Vendredi 6 Janvier 2017 12:51:58 Objet: Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration > >>But I wonder why targetstorage only works

Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration

2017-01-06 Thread Dietmar Maurer
> >>But I wonder why targetstorage only works for online migration?? > > do we allow currently the migrate offline, between different type of storage ? > (zfs->lvm, lvm->qcow2?). No. But my question was more like: why not? I guess this is not really an important feature, because users can migrate

Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration

2017-01-06 Thread Alexandre DERUMIER
nal - De: "dietmar" À: "aderumier" , "pve-devel" Envoyé: Vendredi 6 Janvier 2017 12:25:40 Objet: Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration Applied, thanks! But I wonder why targetstorage only works for online migration??

Re: [pve-devel] [PATCH] add with-local-disks option for live storage migration

2017-01-06 Thread Dietmar Maurer
Applied, thanks! But I wonder why targetstorage only works for online migration?? > On January 6, 2017 at 10:15 AM Alexandre Derumier wrote: > > > As Fabian as required, > add an extra flag "with-local-disks" to enable live storage migration with > localdisk. > > default target storage is s

[pve-devel] [PATCH] add with-local-disks option for live storage migration

2017-01-06 Thread Alexandre Derumier
As Fabian as required, add an extra flag "with-local-disks" to enable live storage migration with localdisk. default target storage is same sid than source, this can be overrided with "targetstorage" option. I will try improve this later, with optionnal mapping, disk by disk. Signed-off-by: Al