[pve-devel] [PATCH container 2/2] cloud-init: add support for multiple IP addresses

2018-10-23 Thread Julian Pawlowski
This will allow to have multiple ip and ip6 definitions in --ifconfig. The first ip and ip6 value will be associated with the network gateway (if present). Julian Pawlowski (1): cloud-init: add support for multiple IP addresses PVE/QemuServer.pm | 50 - PVE/QemuServer/Cloudi

[pve-devel] [PATCH v2 container 2/2] qemu-server: fix default_key check for multi-key support

2018-10-23 Thread Julian Pawlowski
--- src/PVE/JSONSchema.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm index dbc8507..b94aaca 100644 --- a/src/PVE/JSONSchema.pm +++ b/src/PVE/JSONSchema.pm @@ -627,8 +627,8 @@ sub parse_property_string { die "st

[pve-devel] [PATCH container 1/1] cloud-init: add support for multiple IP addresses

2018-10-23 Thread Julian Pawlowski
Signed-off-by: Julian Pawlowski --- PVE/QemuServer.pm | 50 - PVE/QemuServer/Cloudinit.pm | 202 +--- 2 files changed, 145 insertions(+), 107 deletions(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 933f54f..9bd8c18 100644 --- a/PVE

[pve-devel] [PATCH v2 container 1/2] qemu-server: add support for multiple IP addresses

2018-10-23 Thread Julian Pawlowski
Signed-off-by: Julian Pawlowski --- src/PVE/JSONSchema.pm | 97 ++- 1 file changed, 60 insertions(+), 37 deletions(-) diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm index fb58ad3..dbc8507 100644 --- a/src/PVE/JSONSchema.pm +++ b/src/PVE/JSONSch

[pve-devel] [PATCH container 1/2] cloud-init: add support for multiple IP addresses

2018-10-23 Thread Julian Pawlowski
This will prepare the JSON schema to allow using a key multiple times. It will be needed for cloud-init to configure multiple IP addresses to a single network interface. Julian Pawlowski (2): qemu-server: add support for multiple IP addresses qemu-server: fix default_key check for multi-key su

Re: [pve-devel] [PATCH] README: document different behaviour of submodule vs. clone

2018-10-23 Thread Thomas Lamprecht
On 10/24/18 7:19 AM, Fabian Grünbichler wrote: > LGTM in general, some phrasing inline. I assume you tested other ways to > prevent this issue while investigating? e.g., removing the .git entirely > after copying, or setting some kernel build flag? LOCALVERSION can be unset to address this, removi

Re: [pve-devel] [common 1/2] merge pve-snapshot-name and pve-lxc-snapshot-name to pve-snapshot-id.

2018-10-23 Thread Fabian Grünbichler
On Tue, Oct 23, 2018 at 01:25:45PM +0200, Wolfgang Link wrote: > This format was two time declared in PVE::LXC::Config and PVE::QemuServer. > --- > src/PVE/JSONSchema.pm | 17 + > 1 file changed, 17 insertions(+) > > diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm > ind

Re: [pve-devel] [PATCH] README: document different behaviour of submodule vs. clone

2018-10-23 Thread Fabian Grünbichler
LGTM in general, some phrasing inline. I assume you tested other ways to prevent this issue while investigating? e.g., removing the .git entirely after copying, or setting some kernel build flag? On Tue, Oct 23, 2018 at 04:07:31PM +0200, Thomas Lamprecht wrote: > Signed-off-by: Thomas Lamprecht >

Re: [pve-devel] [PATCH manager] add wipe_disk option when destroying ceph disk

2018-10-23 Thread Thomas Lamprecht
On 10/23/18 4:49 PM, Alwin Antreich wrote: > On Tue, Oct 23, 2018 at 04:19:36PM +0200, Thomas Lamprecht wrote: >> On 10/23/18 4:02 PM, Alwin Antreich wrote: >>> Nice, was on my list too. ;) Some comments inline. >>> >>> On Tue, Oct 23, 2018 at 03:33:44PM +0200, David Limbeck wrote: this allows

Re: [pve-devel] [PATCH manager] add wipe_disk option when destroying ceph disk

2018-10-23 Thread Alwin Antreich
Additional, this is also tracked on ceph's side. https://tracker.ceph.com/issues/22354 On Tue, Oct 23, 2018 at 04:49:27PM +0200, Alwin Antreich wrote: > On Tue, Oct 23, 2018 at 04:19:36PM +0200, Thomas Lamprecht wrote: > > On 10/23/18 4:02 PM, Alwin Antreich wrote: > > > Nice, was on my list too.

Re: [pve-devel] [PATCH manager] add wipe_disk option when destroying ceph disk

2018-10-23 Thread Alwin Antreich
On Tue, Oct 23, 2018 at 04:19:36PM +0200, Thomas Lamprecht wrote: > On 10/23/18 4:02 PM, Alwin Antreich wrote: > > Nice, was on my list too. ;) Some comments inline. > > > > On Tue, Oct 23, 2018 at 03:33:44PM +0200, David Limbeck wrote: > >> this allows the disk to be reused as ceph disk by zeroin

Re: [pve-devel] [PATCH manager] add wipe_disk option when destroying ceph disk

2018-10-23 Thread Thomas Lamprecht
On 10/23/18 4:02 PM, Alwin Antreich wrote: > Nice, was on my list too. ;) Some comments inline. > > On Tue, Oct 23, 2018 at 03:33:44PM +0200, David Limbeck wrote: >> this allows the disk to be reused as ceph disk by zeroing the first 200M >> of the destroyed disk >> >> Signed-off-by: David Limbeck

[pve-devel] [PATCH] README: document different behaviour of submodule vs. clone

2018-10-23 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- README | 29 + 1 file changed, 29 insertions(+) diff --git a/README b/README index db997a7..f904269 100644 --- a/README +++ b/README @@ -24,6 +24,35 @@ Additional/Updated Modules: For licensing questions, see: http://open-zfs.org

Re: [pve-devel] [PATCH manager] add wipe_disk option when destroying ceph disk

2018-10-23 Thread Alwin Antreich
Nice, was on my list too. ;) Some comments inline. On Tue, Oct 23, 2018 at 03:33:44PM +0200, David Limbeck wrote: > this allows the disk to be reused as ceph disk by zeroing the first 200M > of the destroyed disk > > Signed-off-by: David Limbeck > --- > PVE/API2/Ceph.pm | 22 +++

[pve-devel] [PATCH manager] add wipe_disk option when destroying ceph disk

2018-10-23 Thread David Limbeck
this allows the disk to be reused as ceph disk by zeroing the first 200M of the destroyed disk Signed-off-by: David Limbeck --- PVE/API2/Ceph.pm | 22 ++ www/manager6/ceph/OSD.js | 18 +- 2 files changed, 39 insertions(+), 1 deletion(-) diff --git a/P

[pve-devel] tunnel replied 'ERR: resume failed - unable to find configuration file for VM 214 - no such machine' to command 'resume 214'

2018-10-23 Thread Stefan Priebe - Profihost AG
Hello, since upgrading to PVE 5 i'm seeing the following error on a regular basis - while stress testing migration (doing 100-200 migrations in a row): 2018-10-23 13:54:42 migration speed: 384.00 MB/s - downtime 113 ms 2018-10-23 13:54:42 migration status: completed 2018-10-23 13:54:42 ERROR: tun

[pve-devel] [common 1/2] merge pve-snapshot-name and pve-lxc-snapshot-name to pve-snapshot-id.

2018-10-23 Thread Wolfgang Link
This format was two time declared in PVE::LXC::Config and PVE::QemuServer. --- src/PVE/JSONSchema.pm | 17 + 1 file changed, 17 insertions(+) diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm index fb58ad3..3f14875 100644 --- a/src/PVE/JSONSchema.pm +++ b/src/PVE/JSONSche

[pve-devel] Preparation to merge multiple snapshot formats.

2018-10-23 Thread Wolfgang Link
At the moment we have three different implementations for snapshots names. pve-snapshot-name, pve-lxc-snapshot-name and direct in pvesm import/export. This patch is the preparation to remove them and only use instead one function 'pve-snapshot-id'. ___

[pve-devel] [common 2/2] Extend the JSON format schema with pve_base_snapshot.

2018-10-23 Thread Wolfgang Link
A base snapshot can be a replication snapshot or a normal snapshot. Normal snapshots use the format configid. --- src/PVE/JSONSchema.pm | 13 + 1 file changed, 13 insertions(+) diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm index 3f14875..d9e4f3d 100644 --- a/src/PVE/JSONS

Re: [pve-devel] [RFC docs/manager 0/9] new doc-generator features

2018-10-23 Thread Thomas Lamprecht
On 10/17/18 1:01 PM, Fabian Grünbichler wrote: > this series extends pve-doc-generator with the following features: > - allow setting include directories for targets generating files via perl XXX > - new rules for generating bash completion files > - new rules for verifying API packages > > as wel

[pve-devel] applied: [PATCH edk2-firmware] build and include AAVMF

2018-10-23 Thread Thomas Lamprecht
On 10/22/18 8:30 PM, Thomas Lamprecht wrote: > To allow use in emulated and native AARCH64 VMs > > Signed-off-by: Thomas Lamprecht > --- > debian/control | 7 +++--- > debian/pve-edk2-firmware.install | 1 + > debian/rules | 40