Re: [pve-devel] [PATCH manager] vzdump mail: Refactor text part

2020-11-17 Thread Dominic Jäger
On Mon, Nov 16, 2020 at 06:12:37PM +0100, Thomas Lamprecht wrote: > > Or move out even more than just the format string generation out, so that it > becomes a simple loop calling > > $text .= render_task_plain($vmid, $task); > > or something similar to that. my $namelength = 20; $text .= sprint

Re: [pve-devel] [PATCH manager] vzdump mail: Refactor text part

2020-11-17 Thread Thomas Lamprecht
On 17.11.20 09:35, Dominic Jäger wrote: > On Mon, Nov 16, 2020 at 06:12:37PM +0100, Thomas Lamprecht wrote: >> >> Or move out even more than just the format string generation out, so that it >> becomes a simple loop calling >> >> $text .= render_task_plain($vmid, $task); >> >> or something similar

[pve-devel] applied: [PATCH manager v2] ui: fix column behavior with browser scaling

2020-11-17 Thread Dominik Csapak
applied, thanks :) ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH widget-toolkit v2] InputPanel: fix column scaling behavior

2020-11-17 Thread Dominik Csapak
some comments inline, looks good otherwise On 11/2/20 3:01 PM, Aaron Lauterer wrote: When scaling the browsers content either via the browser itself or because the OS has a different scaling / DPI setting, it can happen that not all columns have enough space next to each other and thus the last

[pve-devel] [PATCH manager v2] vzdump mail: Refactor text part

2020-11-17 Thread Dominic Jäger
Less lines exeeding the character limit, less nesting, less duplicate code, more readable sprintf arguments. Co-authored-by: Thomas Lamprecht Signed-off-by: Dominic Jäger --- Middle ground sounds good to me :) And separate lines for arguments look nice. PVE/VZDump.pm | 36 +++---

Re: [pve-devel] [PATCH pve-kernel] add patch: vhost: bump max_mem_regions to 509

2020-11-17 Thread Alexandre Derumier
>>What is wrong with setting it as module option? Nothing wrong. in 2016, Dietmar wanted it in kernel. (don't remember, maybe the module option didn't exist at this time) >>above hunk for the submodule shouldn't be included in the patch, it refers to >>a git object which is not in ubuntus tree,

[pve-devel] [PATCH docs] pvecm: explain role of ssh in PVE stack

2020-11-17 Thread Oguz Bektas
add a section describing how SSH tunnels are used in conjunction with PVE. (for #2829) Signed-off-by: Oguz Bektas --- pvecm.adoc | 35 +++ 1 file changed, 35 insertions(+) diff --git a/pvecm.adoc b/pvecm.adoc index 3820c17..10de0a1 100644 --- a/pvecm.adoc +++ b/p

[pve-devel] [RFC/PATCH qemu] fix #3084: fall back to open-iscsi initiatorname

2020-11-17 Thread Fabian Ebner
Fixes vma restore when the target is an iSCSI storage which expects that initiatorname. Also avoids the need to always explicitly set the initiatorname in PVE code, thus fixing moving efidisks from and to such iSCSI storages. Signed-off-by: Fabian Ebner --- The obvious alternative is to add the

Re: [pve-devel] [PATCH widget-toolkit v2] InputPanel: fix column scaling behavior

2020-11-17 Thread Aaron Lauterer
On 11/17/20 10:29 AM, Dominik Csapak wrote: some comments inline, looks good otherwise On 11/2/20 3:01 PM, Aaron Lauterer wrote: When scaling the browsers content either via the browser itself or because the OS has a different scaling / DPI setting, it can happen that not all columns have eno

Re: [pve-devel] [PATCH widget-toolkit v2] InputPanel: fix column scaling behavior

2020-11-17 Thread Dominik Csapak
On 11/17/20 12:41 PM, Aaron Lauterer wrote: On 11/17/20 10:29 AM, Dominik Csapak wrote: some comments inline, looks good otherwise On 11/2/20 3:01 PM, Aaron Lauterer wrote: When scaling the browsers content either via the browser itself or because the OS has a different scaling / DPI setting

Re: [pve-devel] [PATCH widget-toolkit v2] InputPanel: fix column scaling behavior

2020-11-17 Thread Aaron Lauterer
On 11/17/20 1:05 PM, Dominik Csapak wrote: On 11/17/20 12:41 PM, Aaron Lauterer wrote: On 11/17/20 10:29 AM, Dominik Csapak wrote: some comments inline, looks good otherwise On 11/2/20 3:01 PM, Aaron Lauterer wrote: When scaling the browsers content either via the browser itself or [..

[pve-devel] applied: [PATCH manager v2] vzdump mail: Refactor text part

2020-11-17 Thread Thomas Lamprecht
On 17.11.20 11:46, Dominic Jäger wrote: > Less lines exeeding the character limit, less nesting, less duplicate code, > more readable sprintf arguments. > > Co-authored-by: Thomas Lamprecht > Signed-off-by: Dominic Jäger > --- > Middle ground sounds good to me :) And separate lines for arguments

[pve-devel] [PATCH widget-toolkit 2/2 v3] InputPanel: remove old code for useFieldContainer

2020-11-17 Thread Aaron Lauterer
`useFieldContainer` doesn't seem to be used anymore in any of our products: * PVE * PMG * PBS it therefore can be considered dead code. Signed-off-by: Aaron Lauterer --- v3: added newly after discussing on the mailing list that this is probably dead code [0] [0] https://lists.proxmox.com/piperm

[pve-devel] [PATCH widget-toolkit 1/2 v3] InputPanel: fix column scaling behavior

2020-11-17 Thread Aaron Lauterer
When scaling the browsers content either via the browser itself or because the OS has a different scaling / DPI setting, it can happen that not all columns have enough space next to each other and thus the last column is moved further below. This happens especially on chromium bases browsers (e.g.

[pve-devel] BUG Network - MAC - Restore Container

2020-11-17 Thread Detlef Bracker
Dear, a backed up container, they was restored under a new container ID get equal all the networks settings from the orginal backup! When then the admin change only the IP and start then the container, the container they restored, use the SAME MAC! Proxmox start 2 containers with same MAC but

Re: [pve-devel] BUG Network - MAC - Restore Container

2020-11-17 Thread Dominik Csapak
On 11/17/20 10:27 PM, Detlef Bracker wrote: Dear, a backed up container, they was restored under a new container ID get equal all the networks settings from the orginal backup! When then the admin change only the IP and start then the container, the container they restored, use the SAME MAC!