[pve-devel] applied: [PATCH manager 3/9] nodes: startall: correctly handle warning status for delayed task

On 12.05.21 14:32, Fabian Ebner wrote: > Signed-off-by: Fabian Ebner > --- > PVE/API2/Nodes.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > applied, thanks! ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.

[pve-devel] applied: [PATCH manager 4/9] cli: pveam: remove unused private sub

On 12.05.21 14:32, Fabian Ebner wrote: > Signed-off-by: Fabian Ebner > --- > PVE/CLI/pveam.pm | 6 -- > 1 file changed, 6 deletions(-) > > applied, thanks! ___ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/

[pve-devel] applied: [PATCH manager 5/9] cli tools: correctly handle warnings status

On 12.05.21 14:32, Fabian Ebner wrote: > Signed-off-by: Fabian Ebner > --- > PVE/CLI/pveceph.pm | 2 +- > PVE/CLI/pvenode.pm | 2 +- > PVE/CLI/vzdump.pm | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > applied, thanks! ___ pve-devel m

Re: [pve-devel] [PATCH manager 2/9] api/cli: tasks: don't treat warnings status as an error status

On 12.05.21 14:32, Fabian Ebner wrote: > Signed-off-by: Fabian Ebner > --- > > Dependency bump for pve-common is needed. > > PVE/API2/Tasks.pm | 5 - > PVE/CLI/pvenode.pm | 5 - > 2 files changed, 8 insertions(+), 2 deletions(-) > > diff --git a/PVE/API2/Tasks.pm b/PVE/API2/Tasks.pm >

[pve-devel] [PATCH common 2/2] PVE/Tools: add 'upid_get_status_type'

as a single point where we get the type of upid status Signed-off-by: Dominik Csapak --- src/PVE/Tools.pm | 20 1 file changed, 20 insertions(+) diff --git a/src/PVE/Tools.pm b/src/PVE/Tools.pm index 8946e93..7cca4a4 100644 --- a/src/PVE/Tools.pm +++ b/src/PVE/Tools.pm @@ -

[pve-devel] [PATCH common 1/2] PVE/JSONSchema: add pve-task-status-type

to have a format that contains the possible worker task states Signed-off-by: Dominik Csapak --- src/PVE/JSONSchema.pm | 12 1 file changed, 12 insertions(+) diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm index 6297fff..71df690 100644 --- a/src/PVE/JSONSchema.pm +++ b/s

[pve-devel] [PATCH pmg-api 3/3] PMG/API2/Tasks: add statusfilter

like in PVE/PBS Signed-off-by: Dominik Csapak --- src/PMG/API2/Tasks.pm | 31 ++- 1 file changed, 30 insertions(+), 1 deletion(-) diff --git a/src/PMG/API2/Tasks.pm b/src/PMG/API2/Tasks.pm index a369c2a..b3d4279 100644 --- a/src/PMG/API2/Tasks.pm +++ b/src/PMG/API2/T

[pve-devel] [PATCH manager 1/2] PVE/API2/Tasks: add since/until filter for the task list

similar to pbs Signed-off-by: Dominik Csapak --- PVE/API2/Tasks.pm | 15 +++ 1 file changed, 15 insertions(+) diff --git a/PVE/API2/Tasks.pm b/PVE/API2/Tasks.pm index 8f6ab603..8df701e5 100644 --- a/PVE/API2/Tasks.pm +++ b/PVE/API2/Tasks.pm @@ -90,6 +90,16 @@ __PACKAGE__->register_m

[pve-devel] [PATCH manager 2/2] PVE/API2/Tasks: add statusfilter to task list

similar to pbs. the 'errors' filter parameter still overrides this Signed-off-by: Dominik Csapak --- PVE/API2/Tasks.pm | 30 +- 1 file changed, 29 insertions(+), 1 deletion(-) diff --git a/PVE/API2/Tasks.pm b/PVE/API2/Tasks.pm index 8df701e5..eca92715 100644 --- a/PV

[pve-devel] [PATCH pmg-api 1/3] PMG/API2/Tasks: add typefilter

like we have one in PVE/PBS Signed-off-by: Dominik Csapak --- src/PMG/API2/Tasks.pm | 8 1 file changed, 8 insertions(+) diff --git a/src/PMG/API2/Tasks.pm b/src/PMG/API2/Tasks.pm index 17a043e..85edabf 100644 --- a/src/PMG/API2/Tasks.pm +++ b/src/PMG/API2/Tasks.pm @@ -43,6 +43,11 @@ _

[pve-devel] [PATCH pmg-api 2/3] PMG/API2/Tasks: add since and until filter

like in PVE/PBS Signed-off-by: Dominik Csapak --- src/PMG/API2/Tasks.pm | 14 ++ 1 file changed, 14 insertions(+) diff --git a/src/PMG/API2/Tasks.pm b/src/PMG/API2/Tasks.pm index 85edabf..a369c2a 100644 --- a/src/PMG/API2/Tasks.pm +++ b/src/PMG/API2/Tasks.pm @@ -48,6 +48,16 @@ __PAC

[pve-devel] [PATCH common/pve-manager/pmg-api] unify task filter api

this series aims to get some baseline for the task listing filters in the api across products. the most filters are currently in PBS, so we add the missing ones in pve/pmg this changes only the api yet pve-manager patch 2/2 conflicts with Fabian Ebners patch "api/cli: tasks: don't treat warnings

[pve-devel] [PATCH manager 1/2] {qemu, lxc}/Config: adapt to new proxmoxNodeTasks

by adding a preFilter and extraFilter Signed-off-by: Dominik Csapak --- www/manager6/lxc/Config.js | 4 +++- www/manager6/qemu/Config.js | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/www/manager6/lxc/Config.js b/www/manager6/lxc/Config.js index f996bad1..ed0ad2ba 1006

[pve-devel] [PATCH widget-toolkit/manager] unify gui for task listing

ports over the new task listing we have in pbs to widget-toolkit, which is used in pve/pmg for this to work, the previous api series[0] must be applied, otherwise the filter parameter do not exist 0: https://lists.proxmox.com/pipermail/pve-devel/2021-June/048989.html proxmox-widget-toolkit: Dom

[pve-devel] [PATCH widget-toolkit 1/1] node/Tasks: merge improvements from PBS and make it more generic

this copies most of the task grid from pbs, but adds handling so that users can add aribtrary filter fields the filter fields always present are: * since * until * task type * task status other filters fields can be added by giving an 'extraFilter' array which must contain widget definitions that

[pve-devel] [PATCH manager 2/2] ui: node/Config: add vmid filter to task lists

Signed-off-by: Dominik Csapak --- www/manager6/node/Config.js | 8 1 file changed, 8 insertions(+) diff --git a/www/manager6/node/Config.js b/www/manager6/node/Config.js index 235a7480..e33b30d5 100644 --- a/www/manager6/node/Config.js +++ b/www/manager6/node/Config.js @@ -394,6 +394,14

[pve-devel] [PATCH storage] btrfs: check for btrfs in on_add_hook and activate

Signed-off-by: Wolfgang Bumiller --- PVE/Storage/BTRFSPlugin.pm | 30 ++ 1 file changed, 30 insertions(+) diff --git a/PVE/Storage/BTRFSPlugin.pm b/PVE/Storage/BTRFSPlugin.pm index 133edc6..0e111a0 100644 --- a/PVE/Storage/BTRFSPlugin.pm +++ b/PVE/Storage/BTRFSPlugin.

[pve-devel] [PATCH installer] network: sync option names with pve-common

back in 2018, commit 1accc6da659d8a02dcf1c2a636749e6f48fb9c2d "Inotify : write network config : use modern syntax for options" changed pve-common's /etc/network/interfaces writer to prefer options with '-' over those with '_'. having the installer write the old variant makes any modification of th

Re: [pve-devel] applied-series: [PATCH-SERIES v2 manager] stricter storage rules for migration

Am 21.06.21 um 17:18 schrieb Thomas Lamprecht: On 21.06.21 16:31, Fabian Ebner wrote: Changes from v1: * dropped already applied patches * rebased * adapted/improved messages Fabian Ebner (2): pve6to7: add check for guest images on misconfigured storages pve6to7: check for

Re: [pve-devel] [PATCH storage] btrfs: check for btrfs in on_add_hook and activate

On June 24, 2021 9:29 am, Wolfgang Bumiller wrote: > Signed-off-by: Wolfgang Bumiller > --- > PVE/Storage/BTRFSPlugin.pm | 30 ++ > 1 file changed, 30 insertions(+) > > diff --git a/PVE/Storage/BTRFSPlugin.pm b/PVE/Storage/BTRFSPlugin.pm > index 133edc6..0e111a0 10064

[pve-devel] [PATCH container] btrfs: pass size when creating subvolumes

otherwise the exception doesn't make much sense Signed-off-by: Wolfgang Bumiller --- src/PVE/LXC.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/PVE/LXC.pm b/src/PVE/LXC.pm index fc06842..93286f6 100644 --- a/src/PVE/LXC.pm +++ b/src/PVE/LXC.pm @@ -1896,9 +1896,9 @@ su

[pve-devel] [PATCH pve-access-control 4/4] implement OpenID autocreate user feature

--- src/PVE/API2/OpenId.pm | 33 ++--- 1 file changed, 30 insertions(+), 3 deletions(-) diff --git a/src/PVE/API2/OpenId.pm b/src/PVE/API2/OpenId.pm index db9f9eb..3814895 100644 --- a/src/PVE/API2/OpenId.pm +++ b/src/PVE/API2/OpenId.pm @@ -9,9 +9,10 @@ use PVE::RS::Op

[pve-devel] [PATCH pve-manager] ui: implement OpenId login

--- PVE/HTTPServer.pm | 4 +- www/manager6/Utils.js | 8 +++ www/manager6/window/LoginWindow.js | 105 - 3 files changed, 114 insertions(+), 3 deletions(-) diff --git a/PVE/HTTPServer.pm b/PVE/HTTPServer.pm index 636b562b..dabdf7f3 100

[pve-devel] [PATCH pve-access-control 2/4] depend on libpve-rs-perl

--- debian/control | 2 ++ 1 file changed, 2 insertions(+) diff --git a/debian/control b/debian/control index 81a32bd..3ef748b 100644 --- a/debian/control +++ b/debian/control @@ -10,6 +10,7 @@ Build-Depends: debhelper (>= 12~), lintian, perl, libpv

[pve-devel] [PATCH pve-access-control 1/4] add OpenId configuration

--- src/PVE/AccessControl.pm | 2 ++ src/PVE/Auth/Makefile| 3 +- src/PVE/Auth/OpenId.pm | 67 3 files changed, 71 insertions(+), 1 deletion(-) create mode 100755 src/PVE/Auth/OpenId.pm diff --git a/src/PVE/AccessControl.pm b/src/PVE/AccessControl

[pve-devel] [PATCH pve-access-control 3/4] api: implement openid API

This moves compute_api_permission() into RPCEnvironment.pm. --- src/PVE/API2/AccessControl.pm | 60 ++ src/PVE/API2/Makefile | 3 +- src/PVE/API2/OpenId.pm| 214 ++ src/PVE/RPCEnvironment.pm | 49 4 files changed, 273 inserti

[pve-devel] [PATCH storage] btrfs: fix qgroup id and size in volume resize

also fixes volume_size_info in scalar context for subvols Signed-off-by: Wolfgang Bumiller --- PVE/Storage/BTRFSPlugin.pm | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/PVE/Storage/BTRFSPlugin.pm b/PVE/Storage/BTRFSPlugin.pm index 6ae93c2..0d8bf4b 100644 --- a/PVE/Stor

Re: [pve-devel] [PATCH storage] btrfs: check for btrfs in on_add_hook and activate

On Thu, Jun 24, 2021 at 09:56:58AM +0200, Fabian Grünbichler wrote: > On June 24, 2021 9:29 am, Wolfgang Bumiller wrote: > > Signed-off-by: Wolfgang Bumiller > > --- > > PVE/Storage/BTRFSPlugin.pm | 30 ++ > > 1 file changed, 30 insertions(+) > > > > diff --git a/PVE/

[pve-devel] [PATCH v2 storage] btrfs: check for btrfs in on_add_hook and activate

Signed-off-by: Wolfgang Bumiller --- Changes to v1: * Fabian's feedback: * removed on_add_hook since activate_storage is called anyway * moved btrfs check to after the DirPlugin checks PVE/Storage/BTRFSPlugin.pm | 24 +++- 1 file changed, 23 insertions(+), 1 deletio

[pve-devel] applied: [PATCH installer] network: sync option names with pve-common

On 24.06.21 09:29, Fabian Grünbichler wrote: > back in 2018, commit 1accc6da659d8a02dcf1c2a636749e6f48fb9c2d > "Inotify : write network config : use modern syntax for options" > changed pve-common's /etc/network/interfaces writer to prefer options > with '-' over those with '_'. > > having the ins

Re: [pve-devel] [PATCH storage] btrfs: check for btrfs in on_add_hook and activate

On 24.06.21 09:56, Fabian Grünbichler wrote: > On June 24, 2021 9:29 am, Wolfgang Bumiller wrote: >> Signed-off-by: Wolfgang Bumiller >> --- >> PVE/Storage/BTRFSPlugin.pm | 30 ++ >> 1 file changed, 30 insertions(+) >> >> diff --git a/PVE/Storage/BTRFSPlugin.pm b/PVE/S

[pve-devel] applied: [PATCH v2 storage] btrfs: check for btrfs in on_add_hook and activate

On 24.06.21 10:45, Wolfgang Bumiller wrote: > Signed-off-by: Wolfgang Bumiller > --- > Changes to v1: > * Fabian's feedback: > * removed on_add_hook since activate_storage is called anyway > * moved btrfs check to after the DirPlugin checks > > PVE/Storage/BTRFSPlugin.pm | 24 +

Re: [pve-devel] [PATCH storage] btrfs: check for btrfs in on_add_hook and activate

On June 24, 2021 11:10 am, Thomas Lamprecht wrote: > On 24.06.21 09:56, Fabian Grünbichler wrote: >> On June 24, 2021 9:29 am, Wolfgang Bumiller wrote: >>> Signed-off-by: Wolfgang Bumiller >>> --- >>> PVE/Storage/BTRFSPlugin.pm | 30 ++ >>> 1 file changed, 30 insertion

Re: [pve-devel] [PATCH storage] btrfs: fix qgroup id and size in volume resize

On 24.06.21 10:41, Wolfgang Bumiller wrote: > also fixes volume_size_info in scalar context for subvols > > Signed-off-by: Wolfgang Bumiller > --- > PVE/Storage/BTRFSPlugin.pm | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/PVE/Storage/BTRFSPlugin.pm b/PVE/Storage

Re: [pve-devel] [PATCH storage] btrfs: check for btrfs in on_add_hook and activate

On 24.06.21 11:23, Fabian Grünbichler wrote: > On June 24, 2021 11:10 am, Thomas Lamprecht wrote: >> On 24.06.21 09:56, Fabian Grünbichler wrote: >>> On June 24, 2021 9:29 am, Wolfgang Bumiller wrote: sub activate_storage { my ($class, $storeid, $scfg, $cache) = @_; +assert

[pve-devel] applied: [PATCH container] btrfs: pass size when creating subvolumes

On 24.06.21 10:14, Wolfgang Bumiller wrote: > otherwise the exception doesn't make much sense > > Signed-off-by: Wolfgang Bumiller > --- > src/PVE/LXC.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > applied, thanks! ___ pve-devel mai

Re: [pve-devel] [PATCH storage] btrfs: fix qgroup id and size in volume resize

On Thu, Jun 24, 2021 at 11:24:15AM +0200, Thomas Lamprecht wrote: > On 24.06.21 10:41, Wolfgang Bumiller wrote: > > also fixes volume_size_info in scalar context for subvols > > > > Signed-off-by: Wolfgang Bumiller > > --- > > PVE/Storage/BTRFSPlugin.pm | 6 +++--- > > 1 file changed, 3 insertio

[pve-devel] [PATCH storage 3/3] btrfs: fix qgroup id and size in volume resize

also fixes volume_size_info in scalar context for subvols Signed-off-by: Wolfgang Bumiller --- PVE/Storage/BTRFSPlugin.pm | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/PVE/Storage/BTRFSPlugin.pm b/PVE/Storage/BTRFSPlugin.pm index ccdf3c8..f5e5e25 100644 --- a/PVE/Stor

[pve-devel] [PATCH storage 2/3] btrfs: cleanup after qgroups when deleting subvolumes

Signed-off-by: Wolfgang Bumiller --- They literally just get leaked... PVE/Storage/BTRFSPlugin.pm | 34 +- 1 file changed, 29 insertions(+), 5 deletions(-) diff --git a/PVE/Storage/BTRFSPlugin.pm b/PVE/Storage/BTRFSPlugin.pm index 179186b..ccdf3c8 100644 --- a/PV

[pve-devel] [PATCH v3 storage 1/3] btrfs: support quota-based subvols optionally

Signed-off-by: Wolfgang Bumiller --- Difference to the main btrfs series v2: * removed double-eval to catch errors when setting limits in alloc_image Note that the issue that subvols are created with size zero was an issue in pve-container. Volume resizing is fixed in patch 3 PVE/Sto

Re: [pve-devel] [PATCH v3 storage 1/3] btrfs: support quota-based subvols optionally

On June 24, 2021 12:01 pm, Wolfgang Bumiller wrote: > Signed-off-by: Wolfgang Bumiller > --- > Difference to the main btrfs series v2: > * removed double-eval to catch errors when setting limits in > alloc_image > Note that the issue that subvols are created with size zero was an > issue

[pve-devel] [PATCH proxmox-ve] add grub.d snippet

to set distributor and disable os-prober. this allows us to drop grub cfg handling from the installer except for the ZFS on / use case. Signed-off-by: Fabian Grünbichler --- Notes: adding the root=ZFS=.. boot=zfs to the kernel cmdline could also be done in the installer with a snippet -

[pve-devel] applied: [PATCH proxmox-ve] add grub.d snippet

On 24.06.21 12:57, Fabian Grünbichler wrote: > to set distributor and disable os-prober. this allows us to drop grub > cfg handling from the installer except for the ZFS on / use case. > > Signed-off-by: Fabian Grünbichler > --- > > Notes: > adding the root=ZFS=.. boot=zfs to the kernel cmdl

[pve-devel] [PATCH pve-kernel-meta] proxmox-boot: redirect stdout in update-grub snippet

update-grub (via grub-mkconfig) generates the grub configuration by concatenating the output of each snippet (from /etc/grub.d). We need to redirect the output of `proxmox-boot-tool refresh` to not end up with a syntactically wrong config in /boot/grub/grub.cfg (which is not used in any case) qui

[pve-devel] [PATCH qemu-server 2/2] cloud-init: add vendor-data

Vendor data can contain the same config options as user data. When user data is supplied that conflicts with vendor data, the user data is chosen instead. This can help in creating a base for multiple VMs with the same vendor data, but different user data for customization. [0] Support for vendor

[pve-devel] [PATCH qemu-server 1/2] close #3428: remove package_upgrade

Automatically updating packages on boot is not always a good idea, so remove that setting. The `package_upgrade` setting can still be set via snippets if required. Signed-off-by: Mira Limbeck --- PVE/QemuServer/Cloudinit.pm | 2 -- 1 file changed, 2 deletions(-) diff --git a/PVE/QemuServer/Clou

[pve-devel] applied: [PATCH pve-kernel-meta] proxmox-boot: redirect stdout in update-grub snippet

On 24.06.21 13:20, Stoiko Ivanov wrote: > update-grub (via grub-mkconfig) generates the grub configuration by > concatenating the output of each snippet (from /etc/grub.d). > > We need to redirect the output of `proxmox-boot-tool refresh` > to not end up with a syntactically wrong config in /boot/

[pve-devel] [PATCH installer] zfs: allow RAID0 with different-sized disks

zfs_mirror_size_check is already called in get_zfs_raid_setup when necessary, so don't call it unconditionally, as this will cause a false-positive error on RAID0 devices (where size mismatches are fine). Signed-off-by: Stefan Reiter --- proxinstall | 2 -- 1 file changed, 2 deletions(-) diff -

[pve-devel] Proxmox VE 7.0 (beta) released!

Hi all, We are pleased to announce the first beta release of Proxmox Virtual Environment 7.0! The 7.x family is based on the great Debian 11 "Bullseye" and comes with a 5.11 kernel, QEMU 6.0, LXC 4.0, OpenZFS 2.0.4. Note: The current release of Proxmox Virtual Environment 7.0 is a beta version

[pve-devel] [PATCH qemu-server 3/7] add ip options to netdescr

Signed-off-by: Alexandre Derumier --- PVE/QemuServer.pm | 32 1 file changed, 32 insertions(+) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index a5b6fe8..8d1f177 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -910,6 +910,38 @@ my $net_fmt = {

[pve-devel] [PATCH qemu-server 4/7] add vmconfig_update_net_ip on device hotplug

Signed-off-by: Alexandre Derumier --- PVE/QemuServer.pm | 32 1 file changed, 32 insertions(+) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 8d1f177..d1507d6 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -4709,6 +4709,8 @@ sub vmconfig_hotp

[pve-devel] [PATCH qemu-server 2/7] add print_ipconfig

Signed-off-by: Alexandre Derumier --- PVE/QemuServer.pm | 6 ++ 1 file changed, 6 insertions(+) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 25ac052..a5b6fe8 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -1883,6 +1883,12 @@ sub parse_ipconfig { return $res; }

[pve-devel] [PATCH qemu-server 0/7] RFC: sdn: add ipam support

Hi, This is an RFC to implement ipam support on qemu-server. This don't change the cloud-init current working. As I need to manage pending ip registration configuration, to follow the pending netX interfaces. I have added ip,ip6,gw,gw6 field to netX interfaces options. (same than LXC). (as cu

[pve-devel] [PATCH qemu-server 1/7] add ipam module

This is the same code than lxc, it can be move to a common module later Signed-off-by: Alexandre Derumier --- PVE/QemuServer/Ipam.pm | 158 PVE/QemuServer/Makefile | 1 + 2 files changed, 159 insertions(+) create mode 100644 PVE/QemuServer/Ipam.pm di

[pve-devel] [PATCH qemu-server 5/7] add vmconfig_delete_net_ip on device unplug

Signed-off-by: Alexandre Derumier --- PVE/QemuServer.pm | 23 +++ 1 file changed, 23 insertions(+) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index d1507d6..e646503 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -4620,6 +4620,7 @@ sub vmconfig_hotplug_pendi

[pve-devel] [PATCH qemu-server 6/7] api2: add update ip support

Signed-off-by: Alexandre Derumier --- PVE/API2/Qemu.pm | 18 ++ 1 file changed, 18 insertions(+) diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm index 24dba86..e8c7107 100644 --- a/PVE/API2/Qemu.pm +++ b/PVE/API2/Qemu.pm @@ -22,6 +22,7 @@ use PVE::GuestHelpers; use PVE::QemuCon

[pve-devel] [PATCH qemu-server 7/7] api2: add revert ip support

Signed-off-by: Alexandre Derumier --- PVE/API2/Qemu.pm | 3 +++ 1 file changed, 3 insertions(+) diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm index e8c7107..014dee6 100644 --- a/PVE/API2/Qemu.pm +++ b/PVE/API2/Qemu.pm @@ -1275,6 +1275,9 @@ my $update_vm_api = sub { $modif

[pve-devel] [PATCH container] disable btrfs ioctl in user namespaces

Changes the seccomp profile to return EPERM on every btrfs ioctl. Signed-off-by: Wolfgang Bumiller --- src/PVE/LXC.pm | 9 + 1 file changed, 9 insertions(+) diff --git a/src/PVE/LXC.pm b/src/PVE/LXC.pm index 93286f6..393da1b 100644 --- a/src/PVE/LXC.pm +++ b/src/PVE/LXC.pm @@ -435,6 +43

[pve-devel] [PATCH qemu-server 2/2] fix #2175: PVE/API2/Qemu: update_vm_api: check old drive for permissions too

otherwise a user with only VM.Config.CDROM can detach a disk from a VM by updating it to a cdrom drive Signed-off-by: Dominik Csapak --- this is breaking api change, but fits more in line what we would want for the permissions PVE/API2/Qemu.pm | 6 ++ 1 file changed, 6 insertions(+) diff -

[pve-devel] [PATCH qemu-server 1/2] PVE/API2/Qemu/update_vm_api: refactor drive permission check

Signed-off-by: Dominik Csapak --- PVE/API2/Qemu.pm | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm index 005e655..057b8ff 100644 --- a/PVE/API2/Qemu.pm +++ b/PVE/API2/Qemu.pm @@ -1263,6 +1263,17 @@ my $update_vm_

[pve-devel] applied: [PATCH container] disable btrfs ioctl in user namespaces

On 24.06.21 16:09, Wolfgang Bumiller wrote: > Changes the seccomp profile to return EPERM on every btrfs > ioctl. > > Signed-off-by: Wolfgang Bumiller > --- > src/PVE/LXC.pm | 9 + > 1 file changed, 9 insertions(+) > > applied, thanks! ___

[pve-devel] [PATCH qemu-server] don't default to O_DIRECT on btrfs without nocow

otherwise it'll produce a whole lot of checksum errors and while this would be nice as a storage feature check, it's hard to be 100% accurate there anyway since a directory storage can point anywhere, like for instance a btrfs directory, causing the same issue... Signed-off-by: Wolfgang Bumiller

Re: [pve-devel] [PATCH qemu-server] don't default to O_DIRECT on btrfs without nocow

> On 06/24/2021 4:58 PM Wolfgang Bumiller wrote: > > > otherwise it'll produce a whole lot of checksum errors Just a quick note that this can be more refined. For one: it would be nice to have a `volume_has_feature` for this, but for instance the directory storage plugin might not be able t

[pve-devel] [PATCH manager] ui: panel/GuestStatusView: readd necessary xtypes

currently, cbind only applies properties on objects which have an 'xtype' property. commit 38e6634a8fe8c11ec23e555ce55bc1972ccb5ea1 removed those, thinking the xtype in the 'defaults' is enough add them back, noting that cbind does need it, to prevent removal Signed-off-by: Dominik Csapak --- w

[pve-devel] applied: [PATCH qemu-server] don't default to O_DIRECT on btrfs without nocow

On 24.06.21 16:58, Wolfgang Bumiller wrote: > otherwise it'll produce a whole lot of checksum errors > > and while this would be nice as a storage feature check, > it's hard to be 100% accurate there anyway since a directory > storage can point anywhere, like for instance a btrfs > directory, caus

[pve-devel] applied: [PATCH qemu-server 1/2] PVE/API2/Qemu/update_vm_api: refactor drive permission check

On 24.06.21 16:25, Dominik Csapak wrote: > Signed-off-by: Dominik Csapak > --- > PVE/API2/Qemu.pm | 26 +- > 1 file changed, 13 insertions(+), 13 deletions(-) > > applied both patches, thanks! ___ pve-devel mailing list pve-d

[pve-devel] Proxmox 7.0 Beta - Ceph "a password is required" error messages?

Hi, I've setup a new Ceph cluster with the Proxmox 7.0 beta I'm getting notification emails like this: ceph : a password is required ; PWD=/ ; USER=root ; COMMAND=nvme intel > smart-log-add --json /dev/nvme0n1 Do you know if this is expected behaviour, or what it means? Is this some kind of ne