Re: [pve-devel] [PATCH v6 qemu-server 2/6] mtunnel: add API endpoints

2022-10-03 Thread Fabian Grünbichler
On September 30, 2022 1:52 pm, Stefan Hanreich wrote: > > > On 9/28/22 14:50, Fabian Grünbichler wrote: >> the following two endpoints are used for migration on the remote side >> >> POST /nodes/NODE/qemu/VMID/mtunnel >> >> which creates and locks an empty VM config, and spawns the main qmtunne

Re: [pve-devel] [PATCH FOLLOW-UP v6 container 1/3] migration: add remote migration

2022-10-03 Thread Fabian Grünbichler
same as in qemu-server, the following should be squashed into this patch/commit: 8< diff --git a/src/PVE/API2/LXC.pm b/src/PVE/API2/LXC.pm index 4e21be4..3573b59 100644 --- a/src/PVE/API2/LXC.pm +++ b/src/PVE/API2/LXC.pm @@ -2870,7 +2870,7 @@ __PACKAGE__->register_method({

Re: [pve-devel] [PATCH FOLLOW-UP v6 qemu-server 2/6] mtunnel: add API endpoints

2022-10-03 Thread Fabian Grünbichler
as reported by Stefan Hantreich, the following follow-up should be squashed into this patch if applied: 8< diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm index 57083601..4da37678 100644 --- a/PVE/API2/Qemu.pm +++ b/PVE/API2/Qemu.pm @@ -5581,7 +5581,7 @@ __PACKAGE__->register_method({

[pve-devel] [PATCH qemu 1/1] PVE Backup: allow passing max-workers performance setting

2022-10-03 Thread Fiona Ebner
For query-proxmox-support, add an indication that it's possible to use the setting. For now, the other two BackupPerf settings are not exposed: * use-copy-range: would need to be implemented by the backup-dump block driver first, and in fact, the default for backup was changed, because it wasn't

[pve-devel] [PATCH qemu-server 1/1] vzdump: set max-workers QMP option when specified and supported

2022-10-03 Thread Fiona Ebner
If not supported, a warning is printed and the setting is ignored. Signed-off-by: Fiona Ebner --- PVE/VZDump/QemuServer.pm | 30 ++ 1 file changed, 30 insertions(+) diff --git a/PVE/VZDump/QemuServer.pm b/PVE/VZDump/QemuServer.pm index bf0d1c56..7fb51c7d 100644 --- a

[pve-devel] [PATCH docs 4/4] backup: mention max-workers performance setting

2022-10-03 Thread Fiona Ebner
Signed-off-by: Fiona Ebner --- vzdump.adoc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/vzdump.adoc b/vzdump.adoc index 841cf3d..8658e38 100644 --- a/vzdump.adoc +++ b/vzdump.adoc @@ -217,7 +217,8 @@ missed jobs as soon as possible. There are a few settings for tuning ba

[pve-devel] [PATCH docs 1/4] backup: rework storage section, mentioning and recommending PBS

2022-10-03 Thread Fiona Ebner
Signed-off-by: Fiona Ebner --- vzdump.adoc | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/vzdump.adoc b/vzdump.adoc index 115c740..0c5b32e 100644 --- a/vzdump.adoc +++ b/vzdump.adoc @@ -37,12 +37,14 @@ the `vzdump` command line tool. .Backup Storage -Befor

[pve-devel] [PATCH docs 2/4] backup: expand section for jobs

2022-10-03 Thread Fiona Ebner
Signed-off-by: Fiona Ebner --- vzdump.adoc | 24 +--- 1 file changed, 21 insertions(+), 3 deletions(-) diff --git a/vzdump.adoc b/vzdump.adoc index 0c5b32e..01e1c20 100644 --- a/vzdump.adoc +++ b/vzdump.adoc @@ -200,9 +200,27 @@ Backup Jobs --- Besides triggering

[pve-devel] [PATCH guest-common 1/1] vzdump: add 'performance' property string as a setting

2022-10-03 Thread Fiona Ebner
Initially, to be used for tuning backup performance with QEMU. A few users reported IO-related issues during backup after upgrading to PVE 7.x and using a modified QEMU build with max-workers reduced to 8 instead of 16 helped them [0]. Also generalizes the way vzdump property string are handled f

[pve-devel] [PATCH manager 1/1] vzdump: handle new 'performance' property string

2022-10-03 Thread Fiona Ebner
Also generalizes the way vzdump property strings are handled for jobs. Something similar could be done in VZDump.pm, but there the maxfiles and prune-backups settings are currently coupled, so a dedicated parse_performance() is used instead. Can be changed once maxfiles is dropped. Signed-off-by:

[pve-devel] [PATCH-SERIES qemu(-server)/guest-common/manager/docs] make QEMU's max-workers setting configurable as a vzdump setting

2022-10-03 Thread Fiona Ebner
A few users reported IO-related issues during backup after upgrading to PVE 7.x and using a modified QEMU build with max-workers reduced to 8 instead of 16 helped them [0]. Other BackupPerf settings are not exposed, see the qemu patch for why. Introduces a more general 'performance' property stri

[pve-devel] [PATCH docs 3/4] backup: merge sections describing jobs

2022-10-03 Thread Fiona Ebner
keeping a short intro referring to the full section at the beginning. Signed-off-by: Fiona Ebner --- vzdump.adoc | 26 -- 1 file changed, 12 insertions(+), 14 deletions(-) diff --git a/vzdump.adoc b/vzdump.adoc index 01e1c20..841cf3d 100644 --- a/vzdump.adoc +++ b/vzdump