Re: [pve-devel] [PATCH qemu-server] fix call to lock_config and config_file

2016-03-09 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 common] cleanup: regex match group reference

2016-03-09 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] first patch serie for LXC panel

2016-03-09 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 container] fix #912: centos/redhat confusion

2016-03-09 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 container 1/2] config: allow deleting cpuunits and cpulimit

2016-03-09 Thread Dietmar Maurer
applied both patches ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH manager] lxc: always allow blank mac address for autogeneration

2016-03-09 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] [PATCH pve-manager 6/6] fix storage selector 'change' event handling

2016-03-09 Thread Emmanuel Kasper
the initial loading of the storage selector fires a 'change' event via the 'onLoad' method of its parent class at that point the value of the storage selector is empty, and the callback function fails --- www/manager6/lxc/ResourceEdit.js | 3 +++ 1 file changed, 3 insertions(+) diff --git

[pve-devel] [PATCH pve-manager 4/6] copy missing methods from ExtJS 4 Parser.js

2016-03-09 Thread Emmanuel Kasper
--- www/manager6/Parser.js | 142 - 1 file changed, 141 insertions(+), 1 deletion(-) diff --git a/www/manager6/Parser.js b/www/manager6/Parser.js index 5f15a76..68a6b70 100644 --- a/www/manager6/Parser.js +++ b/www/manager6/Parser.js @@ -5,6 +5,16

[pve-devel] [PATCH pve-manager 2/6] load/reload stores of tabpanel children on 'activate' event

2016-03-09 Thread Emmanuel Kasper
--- www/manager6/lxc/DNS.js | 5 ++--- www/manager6/lxc/Network.js | 4 +--- www/manager6/lxc/Options.js | 5 ++--- www/manager6/lxc/Resources.js| 2 +- www/manager6/lxc/SnapshotTree.js | 4 ++-- 5 files changed, 8 insertions(+), 12 deletions(-) diff --git

[pve-devel] [PATCH pve-manager 5/6] Fix ComboBoxes when adding a mountpoint

2016-03-09 Thread Emmanuel Kasper
reasoning identic to: commit 548b29644df81408fc0fb4f3f450047e906c26c7 adapt KVComboBoxes to pass store items using 'comboItems' parameter --- www/manager6/lxc/ResourceEdit.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/www/manager6/lxc/ResourceEdit.js

[pve-devel] [PATCH pve-manager 1/6] fix lxc console selection ComboBox for extjs6

2016-03-09 Thread Emmanuel Kasper
reasoning identic to commit 548b29644df81408fc0fb4f3f450047e906c26c7 adapt KVComboBoxes to pass store items using 'comboItems' parameter commit 7515e62236b9f3c8317c3aa0dbf470e2b8efdd71 ext6migrate fix model behaviour for KVComboBox --- www/manager6/lxc/Options.js | 6 +++--- 1 file changed, 3

[pve-devel] [PATCH pve-manager 3/6] fix display of lxc console button text

2016-03-09 Thread Emmanuel Kasper
same reasoning as 841343e1c80a1d81a0e4fa90def389983f4a60fe do not use applyIf when adding items in initComponent() --- www/manager6/button/ConsoleButton.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/button/ConsoleButton.js

[pve-devel] first patch serie for LXC panel

2016-03-09 Thread Emmanuel Kasper
this patch serie fixes a first group of Lxc tabs ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH qemu-server] parse_vm_config: allow spaces in values

2016-03-09 Thread Wolfgang Bumiller
Ultimately we'll probably want this parser moved to AbstractConfig (and the one in LXC, too), but for now this should fix the problem reported in the re-opened #153. > On March 9, 2016 at 2:51 PM Wolfgang Bumiller wrote: > > > Serial numbers can have spaces (and qemu

[pve-devel] [PATCH qemu-server] parse_vm_config: allow spaces in values

2016-03-09 Thread Wolfgang Bumiller
Serial numbers can have spaces (and qemu space-pads them, but on the right side and sometimes they need to be left-padded). parse_property_string() can work it, so it's already possible to add them, but they'll be stuck in the [pending] section and never applied. --- PVE/QemuServer.pm | 2 +- 1

Re: [pve-devel] [PATCH pve-zsync 1/2] fix #910: Correctly handle undef

2016-03-09 Thread Dietmar Maurer
applied both patches ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH manager] Allow non-self-signed override certificate for API

2016-03-09 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 access-control] Catch error instead of segfaulting

2016-03-09 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] [PATCH qemu-server] fix call to lock_config and config_file

2016-03-09 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- PVE/CLI/qm.pm| 2 +- PVE/VZDump/QemuServer.pm | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/PVE/CLI/qm.pm b/PVE/CLI/qm.pm index 676c5dc..2a2e3c4 100755 --- a/PVE/CLI/qm.pm +++ b/PVE/CLI/qm.pm @@

Re: [pve-devel] [stable-3 qemu-server] fix #909: pass rate to tap_plug()

2016-03-09 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] [stable-3 common] net: tap_plug() needs to apply rate limiting

2016-03-09 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] [stable-3 common] network: tap_rate_limit: fix uninitialized value

2016-03-09 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] [PATCH common] cleanup: regex match group reference

2016-03-09 Thread Wolfgang Bumiller
Don't refer to the regex match group via $1 after running other code. --- src/PVE/Network.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/PVE/Network.pm b/src/PVE/Network.pm index ac2c168..56af2ed 100644 --- a/src/PVE/Network.pm +++ b/src/PVE/Network.pm @@ -539,13

[pve-devel] Problem with vnc ticket timeout

2016-03-09 Thread Eduard Ahmatgareev
Hi. I have small trouble with VNC connect from api to proxmox. I tried run command: root@cluster-2-2:~# pvesh create /nodes/cluster-2-2/qemu/116/vncproxy and after 10 seconds I got next: nc6: connection timed out command '/bin/nc6 -l -p 5900 -w 10 -e '/usr/sbin/qm vncproxy 116 2>/dev/null''

[pve-devel] [PATCH pve-zsync 2/2] Fix variable name

2016-03-09 Thread Fabian Grünbichler
Since $param->{vmid} is never set, this check could not trigger! using the correct $source->{vmid} works as expected --- pve-zsync | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pve-zsync b/pve-zsync index 5fecbc2..212ada9 100644 --- a/pve-zsync +++ b/pve-zsync @@ -484,7

[pve-devel] [PATCH pve-zsync 1/2] fix #910: Correctly handle undef

2016-03-09 Thread Fabian Grünbichler
--- pve-zsync | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/pve-zsync b/pve-zsync index d517ece..5fecbc2 100644 --- a/pve-zsync +++ b/pve-zsync @@ -425,7 +425,7 @@ sub list { $list .= sprintf("%-25s", cut_target_width($name, 25)); $list .=

Re: [pve-devel] [PULL stable-3 firewall 0/6] firewall backports

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

[pve-devel] [PATCH container] fix #912: centos/redhat confusion

2016-03-09 Thread Wolfgang Bumiller
With the introduction of unmanaged containers a check was added to verify that the ostype is the same as the autodetected one. Since our CentOS plugin was named 'redhat' but the ostype setting was 'centos' this error triggered and rendered centos containers unusable. Renamed the

[pve-devel] [PATCH container 1/2] config: allow deleting cpuunits and cpulimit

2016-03-09 Thread Wolfgang Bumiller
These were simply missing. --- src/PVE/LXC/Config.pm | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/PVE/LXC/Config.pm b/src/PVE/LXC/Config.pm index 2f656e5..04ce9ef 100644 --- a/src/PVE/LXC/Config.pm +++ b/src/PVE/LXC/Config.pm @@ -732,6 +732,12 @@ sub update_pct_config {

[pve-devel] [PATCH container 2/2] implement cpulimit hotplugging

2016-03-09 Thread Wolfgang Bumiller
--- src/PVE/LXC/Config.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/PVE/LXC/Config.pm b/src/PVE/LXC/Config.pm index 04ce9ef..7c451ba 100644 --- a/src/PVE/LXC/Config.pm +++ b/src/PVE/LXC/Config.pm @@ -733,7 +733,7 @@ sub update_pct_config { next if

Re: [pve-devel] [PATCH manager] lxc: always allow blank mac address for autogeneration

2016-03-09 Thread Wolfgang Bumiller
Forgot to mention: fixes #911 > On March 9, 2016 at 9:11 AM Wolfgang Bumiller wrote: > > > Same as with KVM. > --- > www/manager/lxc/Network.js | 2 +- > www/manager6/lxc/Network.js | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git

[pve-devel] [PATCH manager] lxc: always allow blank mac address for autogeneration

2016-03-09 Thread Wolfgang Bumiller
Same as with KVM. --- www/manager/lxc/Network.js | 2 +- www/manager6/lxc/Network.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/www/manager/lxc/Network.js b/www/manager/lxc/Network.js index 7d152cc..933a22d 100644 --- a/www/manager/lxc/Network.js +++