Re: [pve-devel] [PATCH] add support for network trunks

2016-01-15 Thread Alexandre DERUMIER
>>So how does an OVS user do the equivalent of >># /sbin/bridge vlan add dev $iface vid 2-4000 ovs-vsctl set port tap171i0 trunk=2,3,4,5,6,7,8,9,...4000 >>I'm sure there's a sane way to do this...? This is the only way... - Mail original - De: "Wolfgang Bumiller"

[pve-devel] [PATCH v2 container 2/3] Check lock for pct start, stop, suspend, shutdown

2016-01-15 Thread Fabian Grünbichler
Correctly error out upon encountering a locked container when doing pct start, pct stop, pct suspend or pct shutdown Signed-off-by: Fabian Grünbichler --- src/PVE/API2/LXC/Status.pm | 12 1 file changed, 12 insertions(+) diff --git

[pve-devel] [PATCH v2 container 1/3] Set backup lock during stop and suspend backups

2016-01-15 Thread Fabian Grünbichler
Correctly set a 'backup' lock while a 'stop' or 'suspend' mode backup is in progress. Signed-off-by: Fabian Grünbichler --- src/PVE/VZDump/LXC.pm | 23 +++ 1 file changed, 23 insertions(+) diff --git a/src/PVE/VZDump/LXC.pm

[pve-devel] [PATCH v2 container 0/3] Set and check locks for container backups

2016-01-15 Thread Fabian Grünbichler
Fixes: #847 Set locks for suspend and stop backups, Check locks in pct start, pct stop, pct shutdown, pct suspend and lxc-pre-start-hook. Fabian Grünbichler (3): Set backup lock during stop and suspend backups Check lock for pct start, stop, suspend, shutdown Check lock in

[pve-devel] [PATCH v2 container 3/3] Check lock in lxc-pre-start-hook script as well

2016-01-15 Thread Fabian Grünbichler
Fixes: #847 Signed-off-by: Fabian Grünbichler --- src/lxc-pve-prestart-hook | 1 + 1 file changed, 1 insertion(+) diff --git a/src/lxc-pve-prestart-hook b/src/lxc-pve-prestart-hook index e88911d..1ba957f 100755 --- a/src/lxc-pve-prestart-hook +++

Re: [pve-devel] [PATCH] add support for network trunks

2016-01-15 Thread Alexandre DERUMIER
>>Given the bridge command syntax we used here: >>system("/sbin/bridge vlan add dev $iface vid 2-4094") == 0 || >> >>And the new command you introduced here: >>+ system("/sbin/bridge vlan add dev $iface vid $trunk") == 0 || >> >>Should this accept ranges via '-'? If so, does OVS also do ranges

Re: [pve-devel] [PATCH] add support for network trunks

2016-01-15 Thread Wolfgang Bumiller
> On January 15, 2016 at 12:02 PM Alexandre DERUMIER > wrote: > > > >>Given the bridge command syntax we used here: > >>system("/sbin/bridge vlan add dev $iface vid 2-4094") == 0 || > >> > >>And the new command you introduced here: > >>+ system("/sbin/bridge vlan add

Re: [pve-devel] [PATCH cluster 1/2] Change CA SSL key size to 4096 bit

2016-01-15 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 v2 container 0/3] Set and check locks for container backups

2016-01-15 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] network : add trunks support

2016-01-15 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] add support for network trunks

2016-01-15 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 v2 container 0/3] Set and check locks for container backups

2016-01-15 Thread Dietmar Maurer
> Fixes: #847 Thought a bit more about it. During backup, we wo PVE::LXC::lock_aquire($vmid); at backup start, and PVE::LXC::lock_release($vmid); when we are done. So instead, we should try to get that lock when we start/stop a VM. The current code does not acquire the lock in start/stop,

Re: [pve-devel] [PATCH] add support for network trunks

2016-01-15 Thread Alexandre DERUMIER
Thanks, I'll try to improve it next week to support network ranges and add support for lxc + gui - Mail original - De: "dietmar" À: "aderumier" , "pve-devel" Envoyé: Vendredi 15 Janvier 2016 17:28:39 Objet: Re:

[pve-devel] [PATCH container 2/3] Check lock for pct start, stop, suspend, shutdown

2016-01-15 Thread Fabian Grünbichler
Correctly error out upon encountering a locked container when doing pct start, pct stop, pct suspend or pct shutdown --- src/PVE/API2/LXC/Status.pm | 12 1 file changed, 12 insertions(+) diff --git a/src/PVE/API2/LXC/Status.pm b/src/PVE/API2/LXC/Status.pm index fb5480d..e119211

[pve-devel] [PATCH container 1/3] Set backup lock during stop and suspend backups

2016-01-15 Thread Fabian Grünbichler
Correctly set a 'backup' lock while a 'stop' or 'suspend' mode backup is in progress. --- src/PVE/VZDump/LXC.pm | 23 +++ 1 file changed, 23 insertions(+) diff --git a/src/PVE/VZDump/LXC.pm b/src/PVE/VZDump/LXC.pm index 7f27f0b..7cd557c 100644 --- a/src/PVE/VZDump/LXC.pm +++

[pve-devel] [PATCH container 3/3] Check lock in lxc-pre-start-hook script as well

2016-01-15 Thread Fabian Grünbichler
--- src/lxc-pve-prestart-hook | 1 + 1 file changed, 1 insertion(+) diff --git a/src/lxc-pve-prestart-hook b/src/lxc-pve-prestart-hook index e88911d..1ba957f 100755 --- a/src/lxc-pve-prestart-hook +++ b/src/lxc-pve-prestart-hook @@ -63,6 +63,7 @@ __PACKAGE__->register_method ({ return

Re: [pve-devel] [PATCH] add support for network trunks

2016-01-15 Thread Wolfgang Bumiller
On Fri, Jan 15, 2016 at 03:15:35AM +0100, Alexandre Derumier wrote: > @@ -454,7 +454,7 @@ my $nic_model_list_txt = join(' ', sort @$nic_model_list); > my $netdesc = { > optional => 1, > type => 'string', format => 'pve-qm-net', > -typetext => "MODEL=XX:XX:XX:XX:XX:XX >

[pve-devel] [PATCH cluster 2/2] Use correct key usage in node SSL certificate

2016-01-15 Thread Fabian Grünbichler
--- data/PVE/Cluster.pm | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/data/PVE/Cluster.pm b/data/PVE/Cluster.pm index d325f8f..27e248f 100644 --- a/data/PVE/Cluster.pm +++ b/data/PVE/Cluster.pm @@ -271,8 +271,7 @@ commonName = $fqdn [ v3_req ] basicConstraints =

[pve-devel] [PATCH cluster 1/2] Change CA SSL key size to 4096 bit

2016-01-15 Thread Fabian Grünbichler
--- data/PVE/Cluster.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/data/PVE/Cluster.pm b/data/PVE/Cluster.pm index 0085316..d325f8f 100644 --- a/data/PVE/Cluster.pm +++ b/data/PVE/Cluster.pm @@ -165,7 +165,7 @@ sub gen_pveca_key { return if -f $pveca_key_fn;

Re: [pve-devel] [PATCH container 1/3] Set backup lock during stop and suspend backups

2016-01-15 Thread Fabian Grünbichler
> Fabian Grünbichler hat am 15. Januar 2016 um > 10:03 geschrieben: > > > Correctly set a 'backup' lock while a 'stop' or 'suspend' > mode backup is in progress. This series should fix #847, with one caveat for a similar situation: While a 'suspend' mode backup

Re: [pve-devel] [PATCH container 1/3] Set backup lock during stop and suspend backups

2016-01-15 Thread Wolfgang Bumiller
On Fri, Jan 15, 2016 at 10:03:31AM +0100, Fabian Grünbichler wrote: > Correctly set a 'backup' lock while a 'stop' or 'suspend' > mode backup is in progress. > --- > src/PVE/VZDump/LXC.pm | 23 +++ > 1 file changed, 23 insertions(+) > > diff --git a/src/PVE/VZDump/LXC.pm