Re: [pve-devel] [PATCH 5/7] Only append volumes which is created by pve-manager

2013-06-23 Thread Alexandre DERUMIER
Maybe this can be done using the regex instead split ? - Mail original - De: m...@datanom.net À: de...@lists.claws-mail.org, pve-devel@pve.proxmox.com Envoyé: Dimanche 23 Juin 2013 22:34:00 Objet: [pve-devel] [PATCH 5/7] Only append volumes which is created by pve-manager From: Mi

Re: [pve-devel] [PATCH 2/7] If remove of a base-image fails because of dependent clones we must recreate the lun since dependent clones require access to the image.

2013-06-23 Thread Alexandre DERUMIER
"If remove of a base-image fails because of dependent clones we must recreate the lun since dependent clones require access to the image." I don't have tested the code yet, (Remap lun on hanging free_image is good), but it's not a problem for clones if the base image is not accessible

[pve-devel] ZFS storage plugin changes to www manager

2013-06-23 Thread mir
This patch makes the needed changes to pvemanagerlib.js to enable support for the ZFS storage plugin. ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH 01/43] Added support for ZFS Storage Plugin

2013-06-23 Thread mir
From: Michael Rasmussen Signed-off-by: Michael Rasmussen --- www/manager/Makefile | 1 + www/manager/Utils.js | 2 + www/manager/dc/StorageView.js | 11 +++ www/manager/qemu/Clone.js | 3 +- www/manager/qemu/HDEdit.js | 3 +- www/manager/storage/ZFSEdit.js

[pve-devel] [PATCH 01/43] Added support for ZFS Storage Plugin

2013-06-23 Thread mir
From: Michael Rasmussen Signed-off-by: Michael Rasmussen --- www/manager/Makefile | 1 + www/manager/Utils.js | 2 + www/manager/dc/StorageView.js | 11 +++ www/manager/qemu/Clone.js | 3 +- www/manager/qemu/HDEdit.js | 3 +- www/manager/storage/ZFSEdit.js

[pve-devel] [PATCH 6/7] Refactored code according to suggestions by Alexandre. Also fixed bug so that parent to an image is returned.

2013-06-23 Thread mir
From: Michael Rasmussen The fix solves timeout problems in pve GUI since refactored code runs 5 times faster than original code. Signed-off-by: Michael Rasmussen --- PVE/Storage/ZFSPlugin.pm | 70 1 file changed, 35 insertions(+), 35 deletions(-

[pve-devel] [PATCH 7/7] Explicite use user root when making ssh call.

2013-06-23 Thread mir
From: Michael Rasmussen Signed-off-by: Michael Rasmussen --- PVE/Storage/ZFSPlugin.pm | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/PVE/Storage/ZFSPlugin.pm b/PVE/Storage/ZFSPlugin.pm index 0ec6632..a562635 100644 --- a/PVE/Storage/ZFSPlugin.pm +++ b/PVE/Stora

[pve-devel] [PATCH 2/7] If remove of a base-image fails because of dependent clones we must recreate the lun since dependent clones require access to the image.

2013-06-23 Thread mir
From: Michael Rasmussen Signed-off-by: Michael Rasmussen --- PVE/Storage/ZFSPlugin.pm | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/PVE/Storage/ZFSPlugin.pm b/PVE/Storage/ZFSPlugin.pm index 909f71f..b02a87a 100644 --- a/PVE/Storage/ZFSPlugin.pm +++ b/PVE/St

[pve-devel] [PATCH 5/7] Only append volumes which is created by pve-manager

2013-06-23 Thread mir
From: Michael Rasmussen Signed-off-by: Michael Rasmussen --- PVE/Storage/ZFSPlugin.pm | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/PVE/Storage/ZFSPlugin.pm b/PVE/Storage/ZFSPlugin.pm index e39b965..ff32b97 100644 --- a/PVE/Storage/ZFSPlugin.pm +++ b/PVE/Storage/ZFSPlu

[pve-devel] [PATCH 4/7] Fix small variable name conflict.

2013-06-23 Thread mir
From: Michael Rasmussen Signed-off-by: Michael Rasmussen --- PVE/Storage/ZFSPlugin.pm | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/PVE/Storage/ZFSPlugin.pm b/PVE/Storage/ZFSPlugin.pm index 0addd0b..e39b965 100644 --- a/PVE/Storage/ZFSPlugin.pm +++ b/PVE/Storage/ZF

[pve-devel] [PATCH 3/7] Remove unnecessary output from error message.

2013-06-23 Thread mir
From: Michael Rasmussen Signed-off-by: Michael Rasmussen --- PVE/Storage/ZFSPlugin.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/Storage/ZFSPlugin.pm b/PVE/Storage/ZFSPlugin.pm index b02a87a..0addd0b 100644 --- a/PVE/Storage/ZFSPlugin.pm +++ b/PVE/Storage/ZFSPlugin.

[pve-devel] [PATCH 1/7] Added support for ZFS Storage Plugin

2013-06-23 Thread mir
From: Michael Rasmussen Signed-off-by: Michael Rasmussen --- PVE/Storage.pm | 2 + PVE/Storage/Makefile | 2 +- PVE/Storage/Plugin.pm| 2 +- PVE/Storage/ZFSPlugin.pm | 646 +++ 4 files changed, 650 insertions(+), 2 deletions(-)

Re: [pve-devel] pveproxy: I might have discovered a bug

2013-06-23 Thread Alexandre DERUMIER
I'm not sure, but maybe it's comming from http_request() from pveproxy to pvedaemon from http://search.cpan.org/~gaas/HTTP-Message-6.06/lib/HTTP/Request.pm " persistent => $boolean Try to create/reuse a persistent connection. When this flag is set (default: true for idempotent requests, false f

Re: [pve-devel] pveproxy: I might have discovered a bug

2013-06-23 Thread Michael Rasmussen
On Sun, 23 Jun 2013 05:55:44 + Dietmar Maurer wrote: > > I also observed repeated (POST) calls last week, but was unable to find the > bug. > Is it possible to assamble an easy test case for that to reproduce the bug? > My best guess is that time is the triggering factor. To produce a test