Re: [pve-devel] Fix Bug 1160 - allow HA templates

2016-11-02 Thread Dietmar Maurer
> Templates could be partly managed by HA, instead of blocking the addition of > template to HA allow them and restrict their functionallity to > migrate/recovery > so that if a node fails the template is available at another node. What if we simply add a 'nostart' flag to the resource instead? _

Re: [pve-devel] [PATCH manager] Add ECDH curves to use with modern ciphers

2016-11-02 Thread Jos Ewert
This patch adds curves to use with TLS_ECDHE_* ciphers They will automatically be used be the proxy as they are in the HIGH ciphersuite. This patch uses the prime256v1 curve, which should be supported by most clients. openssl 1.0.1 only supports a single curve. This also forces the use of new DHE

[pve-devel] applied: [PATCH manager v2 3/4] add pveIntegerfield and use it

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

[pve-devel] applied: [PATCH manager v2 4/4] only allow integers in memoryfield

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

[pve-devel] applied: [PATCH manager v2 1/4] move onlineHelp into classes from definition

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

[pve-devel] applied: [PATCH manager v2 2/4] remove unnecessary onlineHelp property

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

[pve-devel] applied: [PATCH docs] More details in introduction about CD-ROM

2016-11-02 Thread Dietmar Maurer
applied, thanks. But I removed information about exact versions, because this would need to be updated for each release (see below). > On November 2, 2016 at 4:30 PM Friedrich Ramberger > wrote: > > > Signed-off-by: Friedrich Ramberger > --- > pve-installation.adoc | 10 +- > 1 file

[pve-devel] [PATCH manager v2 1/4] move onlineHelp into classes from definition

2016-11-02 Thread Dominik Csapak
this patch moves the onlineHelp ids into the javascript classes, instead of defining them where we use the classes Signed-off-by: Dominik Csapak --- www/manager6/ceph/Config.js | 2 ++ www/manager6/ceph/Monitor.js | 2 ++ www/manager6/ceph/OSD.js | 1 + www/manage

[pve-devel] [PATCH manager v2 2/4] remove unnecessary onlineHelp property

2016-11-02 Thread Dominik Csapak
we already defined it in the inputpanel Signed-off-by: Dominik Csapak --- www/manager6/lxc/CreateWizard.js | 1 - 1 file changed, 1 deletion(-) diff --git a/www/manager6/lxc/CreateWizard.js b/www/manager6/lxc/CreateWizard.js index d3981a9..46f4e81 100644 --- a/www/manager6/lxc/CreateWizard.js +

[pve-devel] [PATCH manager v2 4/4] only allow integers in memoryfield

2016-11-02 Thread Dominik Csapak
Signed-off-by: Dominik Csapak --- www/manager6/form/MemoryField.js | 4 1 file changed, 4 insertions(+) diff --git a/www/manager6/form/MemoryField.js b/www/manager6/form/MemoryField.js index 1c9bca9..bdb2767 100644 --- a/www/manager6/form/MemoryField.js +++ b/www/manager6/form/MemoryField.j

[pve-devel] [PATCH manager v2 3/4] add pveIntegerfield and use it

2016-11-02 Thread Dominik Csapak
this adds a subclass of Ext.form.field.Number with the settings for Integers (allowDecimals: false and allowExponential: false and default step size 1) and use it where we only accept integers Signed-off-by: Dominik Csapak --- www/manager6/Makefile | 1 + www/manager6/ceph/Po

[pve-devel] [RFC ha-manager] Fix #1160: add template resource

2016-11-02 Thread Thomas Lamprecht
This is a very limited resource representing both LXC and QEMU templates. Methods like start, stop and check_running do nothing or tell the LRM want he wants to hear. While templates are a questionable HA resource they are already used as such (see bug 1160) and prohibiting that usage is a draw ba

[pve-devel] [RFC qemu-server] ha: use correct service id if VM is a template

2016-11-02 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- PVE/API2/Qemu.pm | 9 + 1 file changed, 9 insertions(+) diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm index 66dc69a..d790d7c 100644 --- a/PVE/API2/Qemu.pm +++ b/PVE/API2/Qemu.pm @@ -1700,6 +1700,9 @@ __PACKAGE__->register_method({

[pve-devel] [RFC container] ha: use correct service id if CT is a template

2016-11-02 Thread Thomas Lamprecht
while CT templates are experimental still prepare them for HA Signed-off-by: Thomas Lamprecht --- src/PVE/API2/LXC.pm| 5 - src/PVE/API2/LXC/Status.pm | 14 ++ 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/src/PVE/API2/LXC.pm b/src/PVE/API2/LXC.pm index

[pve-devel] Fix Bug 1160 - allow HA templates

2016-11-02 Thread Thomas Lamprecht
Templates could be partly managed by HA, instead of blocking the addition of template to HA allow them and restrict their functionallity to migrate/recovery so that if a node fails the template is available at another node. see https://bugzilla.proxmox.com/show_bug.cgi?id=1160 __

[pve-devel] applied: [PATCH docs] do not require 5 chars for headers

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

[pve-devel] applied: [PATCH manager 5/5] fix jslint errors in ha resources

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

[pve-devel] applied: [PATCH manager 4/5] add lxc cpu and memory onlineHelp ids

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

[pve-devel] applied: [PATCH manager 3/5] correctly hide helpButton in wizard

2016-11-02 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 manager 1/5] move onlineHelp into classes from definition

2016-11-02 Thread Dominik Csapak
On 11/02/2016 04:36 PM, Dietmar Maurer wrote: Applying: move onlineHelp into classes from definition error: patch failed: www/manager6/dc/StorageView.js:4 error: www/manager6/dc/StorageView.js: patch does not apply error: patch failed: www/manager6/grid/FirewallRules.js:377 error: www/manager6/gr

Re: [pve-devel] [PATCH manager 1/2] add pveIntegerfield and use it

2016-11-02 Thread Dominik Csapak
On 11/02/2016 04:28 PM, Dietmar Maurer wrote: why "pveIntegerfield"? Maybe you meant "pveIntegerField"? But extjs does use camel case for xtypes? actually, extjs does not capitalize anything in the shorthand xtype, but only in the full Ext.yyy.Zzz syntax see (http://docs.sencha.com/extjs/6.0.

Re: [pve-devel] [PATCH manager 1/5] move onlineHelp into classes from definition

2016-11-02 Thread Dietmar Maurer
Applying: move onlineHelp into classes from definition error: patch failed: www/manager6/dc/StorageView.js:4 error: www/manager6/dc/StorageView.js: patch does not apply error: patch failed: www/manager6/grid/FirewallRules.js:377 error: www/manager6/grid/FirewallRules.js: patch does not apply Patch

[pve-devel] [PATCH docs] More details in introduction about CD-ROM

2016-11-02 Thread Friedrich Ramberger
Signed-off-by: Friedrich Ramberger --- pve-installation.adoc | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/pve-installation.adoc b/pve-installation.adoc index a309061..ce11c93 100644 --- a/pve-installation.adoc +++ b/pve-installation.adoc @@ -32,11 +32,11 @@ Using

Re: [pve-devel] [PATCH manager 1/2] add pveIntegerfield and use it

2016-11-02 Thread Dietmar Maurer
why "pveIntegerfield"? Maybe you meant "pveIntegerField"? But extjs does use camel case for xtypes? ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH manager] Add ECDH curves to use with modern ciphers

2016-11-02 Thread Fabian Grünbichler
On Wed, Nov 02, 2016 at 03:15:00PM +0100, Jos Ewert wrote: > Hi, > > Thanks for your help. > > I have a few more questions and comments: > > > OpenSSL API methods, which are at least fairly easy to grasp in > > the OpenSSL code base (although the whole ECDH subsystem seems to be > > rather und

[pve-devel] [PATCH manager 1/2] add pveIntegerfield and use it

2016-11-02 Thread Dominik Csapak
this adds a subclass of Ext.form.field.Number with the settings for Integers (allowDecimals: false and allowExponential: false and default step size 1) and use it where we only accept integers Signed-off-by: Dominik Csapak --- www/manager6/Makefile | 1 + www/manager6/ceph/Po

[pve-devel] [PATCH manager 2/2] only allow integers in memoryfield

2016-11-02 Thread Dominik Csapak
Signed-off-by: Dominik Csapak --- www/manager6/form/MemoryField.js | 4 1 file changed, 4 insertions(+) diff --git a/www/manager6/form/MemoryField.js b/www/manager6/form/MemoryField.js index 1c9bca9..bdb2767 100644 --- a/www/manager6/form/MemoryField.js +++ b/www/manager6/form/MemoryField.j

[pve-devel] [PATCH docs] do not require 5 chars for headers

2016-11-02 Thread Dominik Csapak
this patch changes the parses so that it detects headings with at least 3 characters instead of 5 (this prevented some block ids to not have a subchapter e.g. pct CPU) Signed-off-by: Dominik Csapak --- scan-adoc-refs | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/sca

[pve-devel] [PATCH manager 4/5] add lxc cpu and memory onlineHelp ids

2016-11-02 Thread Dominik Csapak
Signed-off-by: Dominik Csapak --- www/manager6/lxc/ResourceEdit.js | 4 1 file changed, 4 insertions(+) diff --git a/www/manager6/lxc/ResourceEdit.js b/www/manager6/lxc/ResourceEdit.js index c9cd4aa..dd1639c 100644 --- a/www/manager6/lxc/ResourceEdit.js +++ b/www/manager6/lxc/ResourceEdit.j

[pve-devel] [PATCH manager 1/5] move onlineHelp into classes from definition

2016-11-02 Thread Dominik Csapak
this patch moves the onlineHelp ids into the javascript classes, instead of defining them where we use the classes Signed-off-by: Dominik Csapak --- www/manager6/ceph/Config.js | 2 ++ www/manager6/ceph/Monitor.js | 2 ++ www/manager6/ceph/OSD.js | 1 + www/manage

[pve-devel] [PATCH manager 5/5] fix jslint errors in ha resources

2016-11-02 Thread Dominik Csapak
also the correct value for haGroupSelector is [] not '', with this, the reset button works correctly Signed-off-by: Dominik Csapak --- www/manager6/ha/GroupSelector.js | 1 + www/manager6/ha/ResourceEdit.js | 1 - www/manager6/ha/Resources.js | 2 +- 3 files changed, 2 insertions(+), 2 dele

[pve-devel] [PATCH manager 3/5] correctly hide helpButton in wizard

2016-11-02 Thread Dominik Csapak
the property is "listenToGlobalEvent" not "listenGlobalEvent" Signed-off-by: Dominik Csapak --- www/manager6/button/HelpButton.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/button/HelpButton.js b/www/manager6/button/HelpButton.js index e038a32..c9bc518 1006

[pve-devel] [PATCH manager 2/5] remove unnecessary onlineHelp property

2016-11-02 Thread Dominik Csapak
we already defined it in the inputpanel Signed-off-by: Dominik Csapak --- www/manager6/lxc/CreateWizard.js | 1 - 1 file changed, 1 deletion(-) diff --git a/www/manager6/lxc/CreateWizard.js b/www/manager6/lxc/CreateWizard.js index d3981a9..46f4e81 100644 --- a/www/manager6/lxc/CreateWizard.js +

Re: [pve-devel] [PATCH common 1/3] tempfile: use /run by default

2016-11-02 Thread Wolfgang Bumiller
Fixes #1188 On Wed, Nov 02, 2016 at 12:25:49PM +0100, Wolfgang Bumiller wrote: > as /tmp is not a tmpfs by default and some file systems > (like ZFS) don't support O_TMPFILE > --- > src/PVE/Tools.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/PVE/Tools.pm b/src/P

[pve-devel] [PATCH common 3/3] tempfile: use /tmp for fallback-tempfiles

2016-11-02 Thread Wolfgang Bumiller
At this point we know it's not a tmpfs (as tmpfs definitely supports O_TMPFILE), so /tmp makes more sense than /run as default path. --- src/PVE/Tools.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/src/PVE/Tools.pm b/src/PVE/Tools.pm index 0be2efe..341f0ac 100644 --- a/src/PVE/Tools.pm +++

[pve-devel] [PATCH common 1/3] tempfile: use /run by default

2016-11-02 Thread Wolfgang Bumiller
as /tmp is not a tmpfs by default and some file systems (like ZFS) don't support O_TMPFILE --- src/PVE/Tools.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/PVE/Tools.pm b/src/PVE/Tools.pm index 7ee3450..305e1bf 100644 --- a/src/PVE/Tools.pm +++ b/src/PVE/Tools.pm @@ -13

[pve-devel] [PATCH common 2/3] tempfile: unliked-file fallback

2016-11-02 Thread Wolfgang Bumiller
some file systems (eg. ZFS) don't support O_TMPFILE --- src/PVE/Tools.pm | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/src/PVE/Tools.pm b/src/PVE/Tools.pm index 305e1bf..0be2efe 100644 --- a/src/PVE/Tools.pm +++ b/src/PVE/Tools.pm @@ -2,7 +2,7 @@ package PVE::Tool

[pve-devel] [PATCH common] JSONSchema: named formats in parse_property_string

2016-11-02 Thread Wolfgang Bumiller
While our code currently passes the format hashes directly, some code gets them via the format property of a configuration description which verifies successfully via check_format() for named property string formats, so this should be allowed. --- src/PVE/JSONSchema.pm | 11 +++ 1 file cha

Re: [pve-devel] pve-qemu-kvm 2.7 : memory state snasphot not working

2016-11-02 Thread Michael Rasmussen
I see it here on ZFS too. On November 2, 2016 10:43:31 AM GMT+01:00, Lindsay Mathieson wrote: >On 2/11/2016 5:28 PM, Alexandre DERUMIER wrote: I checked and the VM state .raw file is only 512 bytes (should be >1+ >>GB), so that is probably the problem. An issue with saving the >snapsho

[pve-devel] applied: [PATCH manager 2/2] forbid decimals and exponent in HA resource edit numberfields

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

[pve-devel] applied: [PATCH manager 1/2] add 'delete_if_default' helper and use it

2016-11-02 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] pve-qemu-kvm 2.7 : memory state snasphot not working

2016-11-02 Thread Lindsay Mathieson
On 2/11/2016 5:28 PM, Alexandre DERUMIER wrote: I checked and the VM state .raw file is only 512 bytes (should be 1+ >>GB), so that is probably the problem. An issue with saving the snapshot. Yes, same here, 512 bytes. For comparison I migrated the VM to a NFS share and tested that - exact sa

Re: [pve-devel] [PATCH manager] Add ECDH curves to use with modern ciphers

2016-11-02 Thread Fabian Grünbichler
On Mon, Oct 31, 2016 at 07:16:41PM +0100, Jos Ewert wrote: > The TLS_ECDHE_* ciphers will automatically be used be the proxy > as they are in the HIGH ciphersuite. > --- > PVE/HTTPServer.pm | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/PVE/HTTPServer.pm b/PVE/HTTPServer.pm > ind

[pve-devel] [PATCH manager 1/2] add 'delete_if_default' helper and use it

2016-11-02 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- www/manager6/Utils.js| 15 +++ www/manager6/ha/ResourceEdit.js | 21 +++-- www/manager6/lxc/ResourceEdit.js | 22 -- www/manager6/qemu/CPUOptions.js | 23 --- 4 files changed, 26

[pve-devel] [PATCH manager 2/2] forbid decimals and exponent in HA resource edit numberfields

2016-11-02 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- www/manager6/ha/ResourceEdit.js | 4 1 file changed, 4 insertions(+) diff --git a/www/manager6/ha/ResourceEdit.js b/www/manager6/ha/ResourceEdit.js index 4ee08a7..4803fc7 100644 --- a/www/manager6/ha/ResourceEdit.js +++ b/www/manager6/ha/ResourceEdit.js @

Re: [pve-devel] pve-qemu-kvm 2.7 : memory state snasphot not working

2016-11-02 Thread Alexandre DERUMIER
>>So maybe it's simply the memory size detection && volume creation which is >>wrong since qemu 2.7 mmm, the volume is first correctly allocated, but shrinked after. qmp query-savevm, give me only after savevm-start: $VAR1 = { 'total-time' => 4583, 'status' => 'active',

Re: [pve-devel] pve-qemu-kvm 2.7 : memory state snasphot not working

2016-11-02 Thread Alexandre DERUMIER
>>I checked and the VM state .raw file is only 512 bytes (should be 1+ >>GB), so that is probably the problem. An issue with saving the snapshot. Yes, same here, 512 bytes. also with ceph, the memorystate is 512bytes. So maybe it's simply the memory size detection && volume creation which is w