[pve-devel] [PATCH v2 manager] fix #3301: show currently running QEMU version

2021-03-09 Thread Fabian Ebner
in the VM summary page. Signed-off-by: Fabian Ebner --- Changes from v1: * use a separate field instead of attaching it directly to the status. Put it directly below the Status field, because it feels like they belong together. OTOH the information might not be as relevant so maybe it

Re: [pve-devel] [PATCH qemu-server 2/2] restore: sanitize config for non-root users

2021-03-09 Thread Fabian Ebner
On 09.03.21 08:17, Thomas Lamprecht wrote: On 08.03.21 13:26, Fabian Ebner wrote: by dropping privileged options for unprivileged users. For better backwards compatibility for in-place restores, keep the option if the value didn't change. Note that this softly "breaks" restoring a backup with s

Re: [pve-devel] [PATCH qemu-server 2/2] restore: sanitize config for non-root users

2021-03-09 Thread Thomas Lamprecht
On 09.03.21 11:08, Fabian Ebner wrote: > On 09.03.21 08:17, Thomas Lamprecht wrote: >> On 08.03.21 13:26, Fabian Ebner wrote: >> >> is this covered by some tests already? >> > > Haven't seen any. I can try and add some tests that mirror the config-related > behavior of the restore_XYZ_archive fun

Re: [pve-devel] [PATCH v2 manager 3/3] fix #2745: ui: backup: allow users to specify remove=1

2021-03-09 Thread Fabian Ebner
On 05.03.21 21:34, Thomas Lamprecht wrote: On 03.03.21 12:50, Fabian Ebner wrote: A user with Datastore.AllocateSpace, VM.Audit, VM.Backup can already remove backups from the GUI manually, so it shouldn't be a problem if they can set the remove flag when starting a manual backup in the GUI. Sig

[pve-devel] [PATCH v6 qemu-server] Add API for import wizards

2021-03-09 Thread Dominic Jäger
Extend qm importdisk/importovf functionality to the API. Signed-off-by: Dominic Jäger --- v5->v6: More parsing Fix regex Improve --boot handling Move readovf from manager to qemu-server (like CPU) Create properties helper for readovf return values PVE/API2/Qemu.pm | 458 +

[pve-devel] [PATCH v6 storage] Optionally allow blockdev in abs_filesystem_path

2021-03-09 Thread Dominic Jäger
This is required to import from LVM storages Signed-off-by: Dominic Jäger --- v5->v6: unchanged PVE/Storage.pm | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/PVE/Storage.pm b/PVE/Storage.pm index 8ee2c92..7c2e24e 100755 --- a/PVE/Storage.pm +++ b/PVE/Storage.pm @@

[pve-devel] [PATCH v6 manager] gui: Add import for disk & VM

2021-03-09 Thread Dominic Jäger
Add GUI wizard to import whole VMs and a window to import single disks in Hardware View. Signed-off-by: Dominic Jäger --- v5->v6: Rebase Move readovf to qemu-server PVE/API2/Nodes.pm | 7 + www/manager6/Makefile | 2 + www/manager6/Workspace.js

Re: [pve-devel] [PATCH v2 manager 3/3] fix #2745: ui: backup: allow users to specify remove=1

2021-03-09 Thread Fabian Ebner
On 09.03.21 11:28, Fabian Ebner wrote: On 05.03.21 21:34, Thomas Lamprecht wrote: On 03.03.21 12:50, Fabian Ebner wrote: A user with Datastore.AllocateSpace, VM.Audit, VM.Backup can already remove backups from the GUI manually, so it shouldn't be a problem if they can set the remove flag when

[pve-devel] avoidable writes of pmxcfs to /var/lib/pve-cluster/config.db ?

2021-03-09 Thread Roland
hello proxmox team, i found that pmxcfs process is quite "chatty" and one of the top disk-writers on our proxmox nodes. i had a closer look, because i was curious, why wearout of our samsung EVO is already at 4% .  as disk I/O of our vms is typically very low, so we used lower end ssd for those

Re: [pve-devel] avoidable writes of pmxcfs to /var/lib/pve-cluster/config.db ?

2021-03-09 Thread Thomas Lamprecht
Hi, On 09.03.21 21:45, Roland wrote: > hello proxmox team, > > i found that pmxcfs process is quite "chatty" and one of the top > disk-writers on our proxmox nodes. > > i had a closer look, because i was curious, why wearout of our samsung > EVO is already at 4% .  as disk I/O of our vms is typi