[pve-devel] [PATCH v3 container 0/2] fix #3443: unique machine-id for containers

2021-05-27 Thread Oguz Bektas
v2->v3: * clear machine-id at the end of clone task worker Oguz Bektas (2): setup: clear /etc/machine-id for newly created containers clear machine-id also after container clone src/PVE/API2/LXC.pm | 6 ++ src/PVE/LXC/Setup.pm | 10 ++ src/PVE/LXC/Setup/Base.pm | 25 +

[pve-devel] [PATCH v3 container 1/2] setup: clear /etc/machine-id for newly created containers

2021-05-27 Thread Oguz Bektas
this way when new containers are created they will have a unique /etc/machine-id Signed-off-by: Oguz Bektas --- v3: no changes src/PVE/LXC/Setup.pm | 10 ++ src/PVE/LXC/Setup/Base.pm | 25 + 2 files changed, 35 insertions(+) diff --git a/src/PVE/LXC/Setup.

[pve-devel] [PATCH v3 container 2/2] clear machine-id also after container clone

2021-05-27 Thread Oguz Bektas
pass $clone=1 to avoid removing the file. instead we truncate it to an empty file Signed-off-by: Oguz Bektas --- v2->v3: * clear machine-id at the end of clone task worker src/PVE/API2/LXC.pm | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/PVE/API2/LXC.pm b/src/PVE/API2/LXC.pm ind

[pve-devel] [PATCH qemu-server 2/2] qm: assume correct VNC setup in 'vncproxy', disallow passwordless

2021-05-27 Thread Stefan Reiter
The QMP 'change' command is no longer available since QEMU 6.0, so this cannot work - instead of replacing it, we can just remove it however. The 'if' branch would only set the VNC socket path anew and enable password mode, which is always set and enabled on startup already. The 'else' branch was

[pve-devel] [PATCH qemu-server 1/2] cfg2cmd: use long form QEMU parameters to avoid warning in 6.0

2021-05-27 Thread Stefan Reiter
QEMU warns us about this: kvm: -chardev socket,id=qmp,path=/var/run/qemu-server/100.qmp,server,nowait: warning: short-form boolean option 'server' deprecated Please use server=on instead kvm: -chardev socket,id=qmp,path=/var/run/qemu-server/100.qmp,server,nowait: warning: short-form boolean opti

[pve-devel] [PATCH pve-qemu 2/3] debian: ignore submodule checks in QEMU build

2021-05-27 Thread Stefan Reiter
...we do those manually, and the build dir is not a git repo. Signed-off-by: Stefan Reiter --- debian/rules | 1 + 1 file changed, 1 insertion(+) diff --git a/debian/rules b/debian/rules index 349c610..1f623d8 100755 --- a/debian/rules +++ b/debian/rules @@ -41,6 +41,7 @@ ${BUILDDIR}/config.sta

[pve-devel] [PATCH pve-qemu 3/3] update keycodemapdb for 6.0

2021-05-27 Thread Stefan Reiter
QEMU 6.0 requires the updated version to build correctly, as the keymap-gen tool gained some new parameters. Signed-off-by: Stefan Reiter --- keycodemapdb/data/keymaps.csv | 16 +- keycodemapdb/meson.build | 1 + keycodemapdb/tests/Makefile | 84 +++ keycodemapdb/tools/keymap-

[pve-devel] [PATCH 0/3] Upgrade to QEMU 6.0

2021-05-27 Thread Stefan Reiter
Mostly smooth migration, so far testing has shown good results. Both my automated tests and some manual ones have shown little issues. Smaller issues encountered so far: * guest agent (especially fs-thaw) still seems to have some trouble, especially on Windows, updating the agent and virtio driv

[pve-devel] applied: [PATCH manager 0/6] fixes/preparation extjs 7.0

2021-05-27 Thread Thomas Lamprecht
On 26.05.21 11:01, Dominik Csapak wrote: > this series contains pve fixes/changes fror extjs7.0 > > all of these patches should be backwards compatible with 6.0.1 > > i tried to test every panel/button with extjs 7.0, > but i am sure i forgot one or the other ;) > > Dominik Csapak (6): > ui: f

[pve-devel] [PATCH manager] ui: ceph/Status: fix icon in status grid

2021-05-27 Thread Dominik Csapak
there was a spurious leftover single quote("'"), that prevented the correct css class to be inserted (e.g., fa-exclamation' instead of fa-exclamation) Signed-off-by: Dominik Csapak --- www/manager6/ceph/Status.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/ce

[pve-devel] [PATCH-SERIES v3 container/qemu-server] fix #3421: allow custom storage plugins to support rootfs

2021-05-27 Thread Lorenz Stechauner
changes to v2: * typo s/supoort/support/ * more detailed error messages * implemented check also for vms pve-container: Lorenz Stechauner (1): fix #3421: allow custom storage plugins to support rootfs src/PVE/LXC.pm | 30 -- 1 file changed, 12 insertions(+), 18 del

[pve-devel] [PATCH v3 container 1/1] fix #3421: allow custom storage plugins to support rootfs

2021-05-27 Thread Lorenz Stechauner
it is now necessary for storages to support the 'rootdir' content in order to start containers on them. all native storage plugins already report the rootdir content correctly. Signed-off-by: Lorenz Stechauner --- src/PVE/LXC.pm | 30 -- 1 file changed, 12 insertions(

[pve-devel] [PATCH v3 qemu-server 1/1] vm_start: check if storages of volumes support content images

2021-05-27 Thread Lorenz Stechauner
it is now necessary for storages to support the 'images' content in order to start vms on them. all native storage plugins already report the images content correctly. Signed-off-by: Lorenz Stechauner --- PVE/QemuServer.pm | 7 +++ 1 file changed, 7 insertions(+) diff --git a/PVE/QemuServer

[pve-devel] [PATCH widget-toolkit] panel/RRDChart: fix animation settings for highlight/marker

2021-05-27 Thread Dominik Csapak
in extjs 7.0, the marker does not have the 'fx' property anymore, but the now documented 'animation' property does not work as it seems. instead set the animation settings directly on the chart. The only downside is that this is now for *all* animations, not only the fade in of the highlight. (the

[pve-devel] [PATCH] [PATCH pve-cluster] SSO feature: login with SAMLv2

2021-05-27 Thread Julien BLAIS
Adding the file /etc/pve/tmp/saml managed by the Auth::SAML plugin Signed-off-by: Julien BLAIS --- data/PVE/Cluster.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/data/PVE/Cluster.pm b/data/PVE/Cluster.pm index 4d09c60..9a45b4f 100644 --- a/data/PVE/Cluster.pm +++ b/data/PVE/Cluster.pm @

[pve-devel] [PATCH] [PATCH pve-access-control] SSO feature: login with SAMLv2

2021-05-27 Thread Julien BLAIS
Added a new endpoint usable by api2/html/access/saml?realm=$DOM which allows to initiate a redirection to an IdP. During initialization, the /etc/pve/tmp/saml file is filled with the format REALM:SAML_REQUEST_ID:TIME Modification of the endpoint /access/ticket to support SAMLResponse. The in

[pve-devel] [PATCH container] [PATCH pve-manager] SSO feature: login with SAMLv2

2021-05-27 Thread Julien BLAIS
Part allowing to add a SAML authentication Adding the rule that the endpoint /access/saml does not need authentication Signed-off-by: Julien BLAIS --- PVE/HTTPServer.pm | 3 +- www/manager6/Makefile | 1 + www/manager6/Utils.js | 5 +++ www/manager6/dc/AuthEd

[pve-devel] [PATCH] [PATCH pve-http-server] SSO feature: login with SAMLv2

2021-05-27 Thread Julien BLAIS
Add a formatted page that allows redirection. For performance reasons, a raw format is used instead of a redirection initiated by the HTTP header. /!\ Modification of the redirection URL for the /access/ticket endpoint. Signed-off-by: Julien BLAIS --- src/PVE/APIServer/Formatter/HTML.pm | 25 +