[pve-devel] [PATCH] Use block storage migration for migration of KVM machines with local based storages

2014-11-03 Thread Kamil Trzcinski
- allow to migrate only VMs with either local or shared storage - for stopped VM start it for migration - allocate remote storage using ssh - enable migration caps: xbzrle and zero_blocks Signed-off-by: Kamil Trzcinski ayu...@ayufan.eu --- PVE/QemuMigrate.pm | 77

[pve-devel] [PATCH] Added methods to allocate and free storage for migration

2014-11-03 Thread Kamil Trzcinski
Signed-off-by: Kamil Trzcinski ayu...@ayufan.eu --- PVE/Storage.pm | 62 ++ 1 file changed, 62 insertions(+) diff --git a/PVE/Storage.pm b/PVE/Storage.pm index 3cc9581..2e3c1ec 100755 --- a/PVE/Storage.pm +++ b/PVE/Storage.pm @@ -477,6

Re: [pve-devel] [PATCH] Use block storage migration for migration of KVM machines with local based storages

2014-11-03 Thread Alexandre DERUMIER
Hi, I didn't known about the blk= option in qmp migrate. Seem to be a lot easier than before. I'll try to test it this week. - enable migration caps: xbzrle and zero_blocks Why do you need xbzrle ? (I'm not sure It's 100% stable yet) - Mail original - De: Kamil Trzcinski

Re: [pve-devel] [PATCH] Use block storage migration for migration of KVM machines with local based storages

2014-11-03 Thread Kamil Trzciński
I'm not sure either. We definitely need zero_blocks :) On Mon, Nov 3, 2014 at 1:24 PM, Alexandre DERUMIER aderum...@odiso.com wrote: Hi, I didn't known about the blk= option in qmp migrate. Seem to be a lot easier than before. I'll try to test it this week. - enable migration caps:

Re: [pve-devel] [PATCH] Use block storage migration for migration of KVM machines with local based storages

2014-11-03 Thread Kamil Trzciński
I didn't known about the blk= option in qmp migrate. It's already for some time. I were doing migrations with blk a two years ago :) ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH] Use block storage migration for migration of KVM machines with local based storages

2014-11-03 Thread Alexandre DERUMIER
Also some notes I have I mind : - We should avoid migration if snapshot exist. (because disk sync will delete snasphots) - We need to sync unused disks - Mail original - De: Kamil Trzciński ayu...@ayufan.eu À: Alexandre DERUMIER aderum...@odiso.com Cc: pve-devel@pve.proxmox.com

Re: [pve-devel] [PATCH] Use block storage migration for migration of KVM machines with local based storages

2014-11-03 Thread Kamil Trzciński
Maybe we should disallow to migrate when either snapshot exist or unused disks are used? On Mon, Nov 3, 2014 at 2:07 PM, Alexandre DERUMIER aderum...@odiso.com wrote: Also some notes I have I mind : - We should avoid migration if snapshot exist. (because disk sync will delete snasphots)

Re: [pve-devel] [PATCH] Use block storage migration for migration of KVM machines with local based storages

2014-11-03 Thread Alexandre DERUMIER
Maybe we should disallow to migrate when either snapshot exist yes or unused disks are used? I'm not sure but maybe it could be possible to use blockdev-add to add the unused disk inside qemu (without exposing it inside the guest with device_add). Like this, maybe it could be possibly work.

Re: [pve-devel] run_command and shellquote

2014-11-03 Thread Michael Rasmussen
On Mon, 3 Nov 2014 05:00:59 + Dietmar Maurer diet...@proxmox.com wrote: Why does shell quoting make the command fail? I thought shell quoting should not have any effect on how the command receives parameters? This is the particular code: my @ssh_opts = ('-o', 'BatchMode=yes'); my

Re: [pve-devel] run_command and shellquote

2014-11-03 Thread Dietmar Maurer
Why does shell quoting make the command fail? I thought shell quoting should not have any effect on how the command receives parameters? This is the particular code: my @ssh_opts = ('-o', 'BatchMode=yes'); my @ssh_cmd = ('/usr/bin/ssh', @ssh_opts); my @params = ('set',