Re: [pve-devel] [PATCH docs] add VLAN explanation.

2017-11-16 Thread Alexandre DERUMIER
Hi, About > + > +auto bond0.5 > +iface bond0.5 inet static > +address 10.10.10.2 > +netmask 255.255.255.0 > +gateway 10.10.10.1 if user try to use vlan5 for his vm, we'll create a vmbrXv5, and enslave bond0.5. but this break network, because ip configuration can't b

Re: [pve-devel] [PATCH cluster 1/6] remove unused sysv init.d service file

2017-11-16 Thread Thomas Lamprecht
On 11/14/2017 02:53 PM, Fabian Grünbichler wrote: > On Mon, Nov 13, 2017 at 12:28:38PM +0100, Thomas Lamprecht wrote: >> Signed-off-by: Thomas Lamprecht >> --- >> debian/init.d | 145 >> - >> debian/pve-cluster.default | 10 >> debia

[pve-devel] [PATCH cluster v2 1/3] pvecm: remove unused variable

2017-11-16 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- data/PVE/CLI/pvecm.pm | 1 - 1 file changed, 1 deletion(-) diff --git a/data/PVE/CLI/pvecm.pm b/data/PVE/CLI/pvecm.pm index 9707114..015978d 100755 --- a/data/PVE/CLI/pvecm.pm +++ b/data/PVE/CLI/pvecm.pm @@ -86,7 +86,6 @@ __PACKAGE__->register_method ({

[pve-devel] [PATCH cluster v2 2/3] pvecm: module cleanup: use our get_host_address_family

2017-11-16 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- data/PVE/CLI/pvecm.pm | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/data/PVE/CLI/pvecm.pm b/data/PVE/CLI/pvecm.pm index 015978d..6b1ab65 100755 --- a/data/PVE/CLI/pvecm.pm +++ b/data/PVE/CLI/pvecm.pm @@ -3,7 +3,6 @@ package PVE::CLI::pve

[pve-devel] [PATCH cluster v2 3/3] pvecm: module cleanup: remove unused modules

2017-11-16 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- data/PVE/CLI/pvecm.pm | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/data/PVE/CLI/pvecm.pm b/data/PVE/CLI/pvecm.pm index 6b1ab65..7448a80 100755 --- a/data/PVE/CLI/pvecm.pm +++ b/data/PVE/CLI/pvecm.pm @@ -2,10 +2,7 @@ package PVE::CLI::

[pve-devel] [PATCH cluster v2 0/3] pvecm: cleanup module usage

2017-11-16 Thread Thomas Lamprecht
cleanup some unused modules and try to use our helpers instead of some other modules. As pvecm is a CLIHandler, which is not used anywhere else (i.e., it's a leaf node from the POV of Perl module dependencies), this is safe to do, there's no risk that we provided a module which was forgotten to im

[pve-devel] [PATCH container 2/2] create: refactor arch detection to run_fork_with_timeout

2017-11-16 Thread Thomas Lamprecht
Swap out our own fork/waitpid code with run_fork_with_timeout, which not only allows to return arbitrary results from the called method but also has a timeout configured, which prevents that a creation hangs forever (= next reboot). As we can now return more than with an exit code number we do not

[pve-devel] [PATCH container 1/2] pct mount: tell user where he can find the mounted CT

2017-11-16 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- src/PVE/CLI/pct.pm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/PVE/CLI/pct.pm b/src/PVE/CLI/pct.pm index 3253906..b82e04b 100755 --- a/src/PVE/CLI/pct.pm +++ b/src/PVE/CLI/pct.pm @@ -297,6 +297,8 @@ __PACKAGE__->register_method({ my $c

Re: [pve-devel] [PATCH docs] add VLAN explanation.

2017-11-16 Thread Thomas Lamprecht
Thanks for this, seems like a good start! Writing down some typos or thoughts I had when reading it, feel free do adopt (or reject, if you think they do not make sense) them. (oh and as said off list, please try to stay below 80 characters per line - where possible) On 11/15/2017 02:55 PM, Wolfg

Re: [pve-devel] LXC IPv6 not set after reboot

2017-11-16 Thread Wolfgang Bumiller
On Wed, Nov 15, 2017 at 01:03:30AM +0100, Bastian Sebode wrote: > Hello, > > I expierience a bug with IPv6 in Containers on PVE 5.1-36. When I set > IPv6 address and IPv6 gateway for the container through the Webinterface > it gets set correctly, but when i reboot the container it doesn't get > se

Re: [pve-devel] [PATCH qemu-server 1/1] Mount storage for VM states before starting VMs from a snapshot

2017-11-16 Thread Fabian Grünbichler
On Mon, Oct 23, 2017 at 02:32:19AM +0200, Katie Holly wrote: > I noticed that when storing VMs and VM snapshot states on a Ceph RBD pool, > Proxmox does not map the volume for the VM state file when restoring from this > snapshot. This patch tries to fix this behavior by mapping the volume for the

[pve-devel] applied: [PATCH docs 1/2] Fix formatting failure

2017-11-16 Thread Wolfgang Bumiller
applied both patches ___ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] applied: [PATCH ha-manager] buildsys: also cleanup *.buildinfo files

2017-11-16 Thread Wolfgang Bumiller
applied On Tue, Nov 07, 2017 at 03:29:01PM +0100, Thomas Lamprecht wrote: > Signed-off-by: Thomas Lamprecht > --- > Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Makefile b/Makefile > index 6f77a08..9778f44 100644 > --- a/Makefile > +++ b/Makefile > @@ -45,7

Re: [pve-devel] [PATCH storage 1/2] Fix: 1542 - show storage utilization per pool, not per global

2017-11-16 Thread Alwin Antreich
On Wed, Nov 15, 2017 at 07:42:57PM +0100, Alexandre DERUMIER wrote: > >>Wouldn't that be totally confusing if you have several pools with different > >>replication factor? > > Personnaly, I don't think it'll be confusing. > Currently you need to do the math manually, and you don't known the > repl

Re: [pve-devel] [PATCH storage 2/2] Addition to #1542, get precent_used from RBDPlugin - add librados2-perl for rados commands - get RBD storage status through librados - calculate %USED if not availa

2017-11-16 Thread Fabian Grünbichler
On Wed, Nov 15, 2017 at 03:47:28PM +0100, Alwin Antreich wrote: > Signed-off-by: Alwin Antreich > --- > PVE/CLI/pvesm.pm | 1 + > PVE/Storage.pm | 6 -- > PVE/Storage/RBDPlugin.pm | 14 +- > 3 files changed, 18 insertions(+), 3 deletions(-) > > diff --git a/PV

Re: [pve-devel] [PATCH storage 1/2] Fix: 1542 - show storage utilization per pool, not per global

2017-11-16 Thread Fabian Grünbichler
On Wed, Nov 15, 2017 at 03:47:27PM +0100, Alwin Antreich wrote: > Signed-off-by: Alwin Antreich > --- > PVE/Storage/RBDPlugin.pm | 40 +++- > 1 file changed, 19 insertions(+), 21 deletions(-) > > diff --git a/PVE/Storage/RBDPlugin.pm b/PVE/Storage/RBDPlugin.pm

[pve-devel] [PATCH qemu-server v2] do not add unused disk when already as unused on another storage

2017-11-16 Thread Dominik Csapak
when having an unused disk on a storage for which there are multiple definitions, we added it again on another storage when that storage was alphabetically before the already existing one this happens for example when using our automatically generated ceph storages: 'pool_ct' and 'pool_vm' and hav