Re: [pve-devel] LXC volumes on ZFS over iSCSI

2016-10-12 Thread Alexandre DERUMIER
>>That and that you have to use hard locking to garanti against data >>loss. The hard locking is the part which can bring any server down if >>the nfs server connection hangs in any way. >>Add to that that it is very difficult to make a high available NFS >>server since client locks are stored in

Re: [pve-devel] [PATCH 1/3] add live storage migration with vm migration

2016-10-12 Thread Thomas Lamprecht
comments inline On 10/11/2016 04:45 PM, Alexandre Derumier wrote: This allow to migrate a local storage (only 1 for now) to a remote node storage. When the target node start, a new volume is created and exposed through qemu embedded nbd server. qemu drive-mirror is done on source vm with nbd

[pve-devel] [PATCH storage] fix #1165: only check mount status when is_mountpoint is set

2016-10-12 Thread Fabian Grünbichler
otherwise the status() method returns garbage for non-mount point directory storages. --- PVE/Storage/DirPlugin.pm | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/PVE/Storage/DirPlugin.pm b/PVE/Storage/DirPlugin.pm index 39f3b30..629fa17 100644 ---

[pve-devel] [PATCH v2 pve-docs] Add useful examples to create/resize a lvm[-thin] pool.

2016-10-12 Thread Wolfgang Link
--- local-lvm.adoc | 63 ++ 1 file changed, 63 insertions(+) diff --git a/local-lvm.adoc b/local-lvm.adoc index 45e875f..603e23d 100644 --- a/local-lvm.adoc +++ b/local-lvm.adoc @@ -41,6 +41,12 @@ data:: This volume uses LVM-thin, and is

[pve-devel] [PATCH v2 common] SectionConfig: always write out explicitly set booleans

2016-10-12 Thread Wolfgang Bumiller
Because when only writing the ones which are true we're breaking the ones which default to true (like the mkdir option on directory storages, where we need a false value to be written out explicitly). --- Changes to v1: Only write out explicitly set values. This is nicer as values which are

[pve-devel] [RFC manager 2/2] add 'my settings' to logout button

2016-10-12 Thread Dominik Csapak
this adds a menu below 'logout' with an entry 'my settings' this opens a window with (for now) browser specific settings and a place to reset them the plan would be to add password change, 2fa setup etc here one remark: the reset layout button reloads the window, to really apply the reset

[pve-devel] [RFC manager 0/2] client settings

2016-10-12 Thread Dominik Csapak
this patch series adds a 'my settings' window and enables saving the user name on login my question, is this useful? currently you can only reset the saved username, and the layout when we have a 'my settings' window, we can add several things there: user preferences (e.g. default storages?)

[pve-devel] [RFC manager 1/2] make login user name saveable

2016-10-12 Thread Dominik Csapak
this patch adds a checkbox to the login window, which uses the localstorage to save the username if one is saved, set the textfield and move focus to the password field Signed-off-by: Dominik Csapak --- www/manager6/window/LoginWindow.js | 42

[pve-devel] [PATCH manager 1/3] Turn PVE.Utils into a singleton object

2016-10-12 Thread Emmanuel Kasper
This will allow us to add contructed values ( regexp a = stringA + stringB ) into the constructor. Access to the PVE.Utils properties and methods stays the same. The double curly brace at the end is due to a double opening curly brace when doing the declaration with Ext.define Not sure if

[pve-devel] [PATCH manager 2/3] Move globally defined regexpes into PVE.Utils

2016-10-12 Thread Emmanuel Kasper
Those globally defined regexpes were only used in Toolkit.js --- www/manager6/Toolkit.js | 20 +++--- www/manager6/Utils.js | 70 - 2 files changed, 44 insertions(+), 46 deletions(-) diff --git a/www/manager6/Toolkit.js

[pve-devel] [PATCH manager] fix 'NaN%' when max value is 0

2016-10-12 Thread Dominik Csapak
when we get a max value of 0 (for example when a storage is not active) the render function would show: 'NaN% (0B of 0B)' because of a division by 0 this patch simply returns 'N/A' because a max value of 0 should never be valid Signed-off-by: Dominik Csapak ---

[pve-devel] [PATCH manager 3/3] Fix jslint warning about trailing comma

2016-10-12 Thread Emmanuel Kasper
--- www/manager6/qemu/OSDefaults.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/qemu/OSDefaults.js b/www/manager6/qemu/OSDefaults.js index f2fc8f4..09190e0 100644 --- a/www/manager6/qemu/OSDefaults.js +++ b/www/manager6/qemu/OSDefaults.js @@ -71,7 +71,7 @@

Re: [pve-devel] [PATCH v2 pve-docs] Add useful examples to create/resize a lvm[-thin] pool.

2016-10-12 Thread Fabian Grünbichler
comments inline On Wed, Oct 12, 2016 at 12:07:13PM +0200, Wolfgang Link wrote: > --- > local-lvm.adoc | 63 > ++ > 1 file changed, 63 insertions(+) > > diff --git a/local-lvm.adoc b/local-lvm.adoc > index 45e875f..603e23d 100644 > ---

[pve-devel] [PATCH v3 pve-docs] Add useful examples to create/resize a lvm[-thin] pool.

2016-10-12 Thread Wolfgang Link
--- local-lvm.adoc | 69 ++ 1 file changed, 69 insertions(+) diff --git a/local-lvm.adoc b/local-lvm.adoc index 45e875f..4c276c1 100644 --- a/local-lvm.adoc +++ b/local-lvm.adoc @@ -41,6 +41,12 @@ data:: This volume uses LVM-thin, and is

[pve-devel] [PATCH manager 1/2] vzdump: document restricted API parameters

2016-10-12 Thread Fabian Grünbichler
--- PVE/API2/VZDump.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/API2/VZDump.pm b/PVE/API2/VZDump.pm index 2e90078..6a76096 100644 --- a/PVE/API2/VZDump.pm +++ b/PVE/API2/VZDump.pm @@ -23,7 +23,7 @@ __PACKAGE__->register_method ({ method => 'POST',

[pve-devel] [PATCH manager 2/2] vzdump: move restricted API parameters check

2016-10-12 Thread Fabian Grünbichler
for better visibility and earlier error detection --- PVE/API2/VZDump.pm | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/PVE/API2/VZDump.pm b/PVE/API2/VZDump.pm index 6a76096..8767aa2 100644 --- a/PVE/API2/VZDump.pm +++ b/PVE/API2/VZDump.pm @@ -56,6 +56,11 @@

Re: [pve-devel] [PATCH manager 1/2] vzdump: document restricted API parameters

2016-10-12 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] qemu2.7: cpu hotplug && hot-unplug v1

2016-10-12 Thread Wolfgang Bumiller
FYI a first 2.7 build is on pvetest now On Tue, Sep 06, 2016 at 12:41:52PM +0200, Alexandre Derumier wrote: > Hi, > This patch series implement new qemu 2.7 cpu hotplug framework, with > device_add|device_del > > hot-unplug is working now ! > > config example: > --- > cpu: kvm64 >

Re: [pve-devel] [PATCH 3/4] cpu hotplug : add cpu hot-unplug support

2016-10-12 Thread Wolfgang Bumiller
On Tue, Sep 06, 2016 at 12:41:55PM +0200, Alexandre Derumier wrote: > Signed-off-by: Alexandre Derumier > --- > PVE/QemuServer.pm | 31 +-- > 1 file changed, 29 insertions(+), 2 deletions(-) > > diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm

Re: [pve-devel] [PATCH 3/4] cpu hotplug : add cpu hot-unplug support

2016-10-12 Thread Alexandre DERUMIER
> +my $machine_type = PVE::QemuServer::get_current_qemu_machine($vmid); > +my $kvmver = kvm_user_version(); >>Note that this calls out to the qemu binary rather than querying the >>currently running qemu process, which is probably what you meant to >>query instead. >>We could add an

Re: [pve-devel] qemu2.7: cpu hotplug && hot-unplug v1

2016-10-12 Thread Alexandre DERUMIER
>>FYI a first 2.7 build is on pvetest now Thanks Wolfgang ! I'll retry my cpu hotplug patches tommorow - Mail original - De: "Wolfgang Bumiller" À: "aderumier" Cc: "pve-devel" Envoyé: Mercredi 12 Octobre 2016

Re: [pve-devel] [PATCH docs] sysadmin: add section about NTP configuration

2016-10-12 Thread Dietmar Maurer
applied, thanks! ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH manager 2/2] vzdump: move restricted API parameters check

2016-10-12 Thread Dietmar Maurer
applied ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH 1/3] add live storage migration with vm migration

2016-10-12 Thread Alexandre DERUMIER
> +#if storage migration is already done, but vm migration crash, we need > to move the vm config >>Here the VM is not always crashed, just the migration failed or? >> >>If yes, would it not be better to let the VM continue to run where it >>was (if it can even run here) and free the

[pve-devel] [PATCH pve-docs] Merge Wiki Network Model in docs

2016-10-12 Thread Wolfgang Link
The most part are already include pve-network.adoc only VLAN are missing. The NAT QEMU redirection goes in the qm.adoc --- pve-network.adoc | 52 qm.adoc | 26 ++ 2 files changed, 78 insertions(+) diff --git