Re: [pve-devel] novnc : revert to 8f3c0f6b9b5e5c23a7dc7e90bd22901017ab4fc7

2015-08-31 Thread Martin Maurer
On 31.08.2015 14:01, Alexandre Derumier wrote: Hi, Martin reported garbage display on novnc. Seem that it's occur since this commit https://github.com/kanaka/noVNC/commit/abf2b09ea7ca59d006c250ea96c5f219f83ca7d0 after introduction of optimization and new inflator.js code. So, it's better to

Re: [pve-devel] [PATCH] HA parse_sid changed to accept CT

2015-08-31 Thread Daniel Hunsaker
Also note that "colon" (:) only has one "l"... On Mon, Aug 31, 2015, 07:48 Alen Grizonic wrote: > --- > src/PVE/HA/Tools.pm | 15 +++ > 1 file changed, 11 insertions(+), 4 deletions(-) > > diff --git a/src/PVE/HA/Tools.pm b/src/PVE/HA/Tools.pm > index

[pve-devel] [PATCH pve-manager] ext5migrate: adapt Backup items to new KVCombobox parent class

2015-08-31 Thread Emmanuel Kasper
This allows the Backup to work with ExtJS5 --- www/manager5/form/BackupModeSelector.js| 17 + www/manager5/form/CompressionSelector.js | 17 + www/manager5/form/DayOfWeekSelector.js | 11 ++-

[pve-devel] [PATCH pve-storage 1/2] iSCSI ipv6 support

2015-08-31 Thread Wolfgang Bumiller
--- PVE/Storage.pm | 9 - PVE/Storage/ISCSIPlugin.pm | 12 +--- PVE/Storage/LunCmd/Iet.pm | 8 PVE/Storage/Plugin.pm | 2 +- 4 files changed, 14 insertions(+), 17 deletions(-) diff --git a/PVE/Storage.pm b/PVE/Storage.pm index d55b645..927219a 100755

[pve-devel] [PATCH pve-storage 2/2] GlusterfsPlugin: replace Net::Ping with tcp_ping

2015-08-31 Thread Wolfgang Bumiller
Using PVE::Network::tcp_ping on port 7 (echo) for ipv6 support. --- PVE/Storage/GlusterfsPlugin.pm | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/PVE/Storage/GlusterfsPlugin.pm b/PVE/Storage/GlusterfsPlugin.pm index 7a8c82f..e59f1ee 100644 ---

Re: [pve-devel] novnc update

2015-08-31 Thread Alexandre DERUMIER
>>I just tested it, does not really work (shows garbage). see screenshot >>attached. tested on IE11, FF, Chrome - all on win7. Don't have problem here (archlinux - firefox/chrome). Is it windows specific ? - Mail original - De: "Martin Maurer" À: "pve-devel"

Re: [pve-devel] [PATCH pve-manager] set size in the rootfs parameter on CT create

2015-08-31 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] novnc update

2015-08-31 Thread Alexandre DERUMIER
ok, I can reproduce it now. I'll try to bissect this - Mail original - De: "Martin Maurer" À: "aderumier" Cc: "pve-devel" Envoyé: Lundi 31 Août 2015 12:22:33 Objet: Re: [pve-devel] novnc update On 31.08.2015 12:12,

Re: [pve-devel] novnc update

2015-08-31 Thread Alexandre DERUMIER
It's working fine for me at this commit (fullscreen support) https://github.com/kanaka/noVNC/commit/340290fa1d6a53d30a38b6604bcacef5940c4c52 but garbage occur since this optimisation commit https://github.com/kanaka/noVNC/commit/abf2b09ea7ca59d006c250ea96c5f219f83ca7d0 I'll make a patch to

Re: [pve-devel] [PATCH pve-manager] set size in the rootfs parameter on CT create

2015-08-31 Thread Thomas Lamprecht
Would be nice to be applied, as at the moment it's impossible to create containers from the manager with the newest packages. Only reminding in case it was lost/forgotten with all the patches lately :) On 08/27/2015 12:13 PM, Thomas Lamprecht wrote: As the size parameter is now deprecated

Re: [pve-devel] novnc update

2015-08-31 Thread Martin Maurer
On 31.08.2015 12:12, Alexandre DERUMIER wrote: I just tested it, does not really work (shows garbage). see screenshot attached. tested on IE11, FF, Chrome - all on win7. Don't have problem here (archlinux - firefox/chrome). Is it windows specific ? No, we also see this on latest FF on

Re: [pve-devel] [PATCH pve-container] skip lxc hooks on non-PVE containers

2015-08-31 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-common] added 'extra-args' CLI handling

2015-08-31 Thread Wolfgang Bumiller
added 'extra-args' standard option added 'extra-args' handling to PVE::JSONSchema::get_options untainting 'extra-args' separately in RESTHandler::handle --- src/PVE/JSONSchema.pm | 16 ++-- src/PVE/RESTHandler.pm | 2 ++ 2 files changed, 16 insertions(+), 2 deletions(-) diff --git

Re: [pve-devel] [PATCH pve-container] skip lxc hooks on non-PVE containers

2015-08-31 Thread Wolfgang Bumiller
> You can extract the bridge name from the LXC configuration? See > > # man lxc.container.conf > > lxc.network.link lxc does that on its own ___ pve-devel mailing list pve-devel@pve.proxmox.com

[pve-devel] [RFC] Adding PVECT resource class so that CT can be HA managed

2015-08-31 Thread Thomas Lamprecht
Adding an abstraction so that the HA manager can start, stop and migrate services independent of the its type. Signed-off-by: Thomas Lamprecht --- src/PVE/HA/Config.pm| 1 + src/PVE/HA/Env/PVE2.pm | 55 ++ src/PVE/HA/Resources.pm | 121

[pve-devel] novnc : revert to 8f3c0f6b9b5e5c23a7dc7e90bd22901017ab4fc7

2015-08-31 Thread Alexandre Derumier
Hi, Martin reported garbage display on novnc. Seem that it's occur since this commit https://github.com/kanaka/noVNC/commit/abf2b09ea7ca59d006c250ea96c5f219f83ca7d0 after introduction of optimization and new inflator.js code. So, it's better to revert to commit

[pve-devel] [PATCH] pve-manager : novncindex : remove include inflator.js

2015-08-31 Thread Alexandre Derumier
Signed-off-by: Alexandre Derumier --- PVE/NoVncIndex.pm | 1 - 1 file changed, 1 deletion(-) diff --git a/PVE/NoVncIndex.pm b/PVE/NoVncIndex.pm index 3dda9aa..d17a003 100644 --- a/PVE/NoVncIndex.pm +++ b/PVE/NoVncIndex.pm @@ -171,7 +171,6 @@ INCLUDE_URI='/novnc/include/';

[pve-devel] [PATCH] remove inflator.js

2015-08-31 Thread Alexandre Derumier
Signed-off-by: Alexandre Derumier --- debian/install | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/debian/install b/debian/install index 8778541..b9bf508 100644 --- a/debian/install +++ b/debian/install @@ -19,4 +19,4 @@ include/jsunzip.js

[pve-devel] [PATCH pve-container] underscore is a valid lxc config key character

2015-08-31 Thread Wolfgang Bumiller
--- src/PVE/LXC.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/PVE/LXC.pm b/src/PVE/LXC.pm index 718fcbf..56d4fcb 100644 --- a/src/PVE/LXC.pm +++ b/src/PVE/LXC.pm @@ -378,7 +378,7 @@ sub parse_pct_config { next; } - if ($line =~

[pve-devel] [PATCH pve-storage] Correct manpage examples to newer syntax

2015-08-31 Thread Emmanuel Kasper
--- pvesm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pvesm b/pvesm index 60e71dc..6c16568 100755 --- a/pvesm +++ b/pvesm @@ -232,10 +232,10 @@ To get the filesystem path for a use: =head1 EXAMPLES # scan iscsi host for available targets - pvesm scan iscsi

[pve-devel] [PATCH pve-container] skip lxc hooks on non-PVE containers

2015-08-31 Thread Wolfgang Bumiller
Our containers all only have a number as name. If any other character appears in a container name, skip our hooks in order to allow lxc to be used manually without interference from PVE. --- src/lxc-pve-mount-hook| 3 +++ src/lxc-pve-poststop-hook | 3 +++ src/lxc-pve-prestart-hook | 3 +++

[pve-devel] [PATCH] HA parse_sid changed to accept CT

2015-08-31 Thread Alen Grizonic
--- src/PVE/HA/Tools.pm | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/src/PVE/HA/Tools.pm b/src/PVE/HA/Tools.pm index 94613de..2b9ca6c 100644 --- a/src/PVE/HA/Tools.pm +++ b/src/PVE/HA/Tools.pm @@ -5,6 +5,7 @@ use warnings; use JSON; use PVE::JSONSchema;

Re: [pve-devel] [PATCH pve-container] skip lxc hooks on non-PVE containers

2015-08-31 Thread Dietmar Maurer
but we still want to add the veth device to the bridge? > diff --git a/src/lxcnetaddbr b/src/lxcnetaddbr > index bb8d726..aca6191 100755 > --- a/src/lxcnetaddbr > +++ b/src/lxcnetaddbr > @@ -3,6 +3,8 @@ > use strict; > use warnings; > > +exit 0 if $ENV{LXC_NAME} !~ /^\d+$/; > + > use

Re: [pve-devel] [PATCH pve-storage 1/2] iSCSI ipv6 support

2015-08-31 Thread Dietmar Maurer
applied both patches, thanks. ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH pve-container] skip lxc hooks on non-PVE containers

2015-08-31 Thread Wolfgang Bumiller
> but we still want to add the veth device to the bridge? not when starting a custom non pve managed container via lxc-start (there wouldn't be a container configuration file available to extract a bridge name from) ___ pve-devel mailing list

[pve-devel] [PATCH] lxc detect error message changed

2015-08-31 Thread Alen Grizonic
--- src/PVE/API2/LXC.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/PVE/API2/LXC.pm b/src/PVE/API2/LXC.pm index 2c42622..7411bdd 100644 --- a/src/PVE/API2/LXC.pm +++ b/src/PVE/API2/LXC.pm @@ -311,7 +311,7 @@ __PACKAGE__->register_method({ my