[pve-devel] applied: [PATCH ja.po 1/1] Signed-off-by: ribbon

2021-07-01 Thread Thomas Lamprecht
On 01.07.21 11:11, ribbon wrote: > update ja.po for proxmox-ve 7.0 > --- > ja.po | 895 ++ > 1 file changed, 335 insertions(+), 560 deletions(-) > > applied, much thanks! ___ pve-devel mailing l

[pve-devel] applied: [PATCH proxmox-apt] Support quoted parsing for .list format

2021-07-01 Thread Thomas Lamprecht
On 01.07.21 15:46, Fabian Ebner wrote: > First patch is just replacing the unicode back-tick that caused > problems down the line. Not related to the second patch. > > Second patch makes the .lists files parser aware of quoting, which > fixes parsing CD ROM and file:// repositories with spaces, an

[pve-devel] [PATCH proxmox-apt] Support quoted parsing for .list format

2021-07-01 Thread Fabian Ebner
First patch is just replacing the unicode back-tick that caused problems down the line. Not related to the second patch. Second patch makes the .lists files parser aware of quoting, which fixes parsing CD ROM and file:// repositories with spaces, and in general makes the parser behave more like AP

[pve-devel] [PATCH proxmox-apt 2/2] support quote-word parsing for one-line format

2021-07-01 Thread Fabian Ebner
so that parsing CD ROM repositories with spaces in the name works too. But it's not limited to that, and should make one-line parsing rather similar to what APT does (stanza parsing in APT doesn't use ParseQuoteWord at all AFAICS). Signed-off-by: Fabian Ebner --- src/repositories/file/list_parse

[pve-devel] [PATCH proxmox-apt 1/2] avoid backtick unicode symbol in string

2021-07-01 Thread Fabian Ebner
Signed-off-by: Fabian Ebner --- src/repositories/standard.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/repositories/standard.rs b/src/repositories/standard.rs index 9d48670..463e735 100644 --- a/src/repositories/standard.rs +++ b/src/repositories/standard.rs @@ -110,

[pve-devel] [PATCH ja.po 1/1] Signed-off-by: ribbon

2021-07-01 Thread ribbon
update ja.po for proxmox-ve 7.0 --- ja.po | 895 ++ 1 file changed, 335 insertions(+), 560 deletions(-) diff --git a/ja.po b/ja.po index 9cac23e..6d4dfd1 100644 --- a/ja.po +++ b/ja.po @@ -35,7 +35,7 @@ msgstr "/some/path" #: pmg-gui/js/P

[pve-devel] SPAM: [PATCH ja.po 0/1] ja.po patch for proxmox-ve 7.0

2021-07-01 Thread ribbon
From: ribbon *** BLURB HERE *** ribbon (1): Signed-off-by: ribbon ja.po | 895 ++ 1 file changed, 335 insertions(+), 560 deletions(-) -- 2.26.2 ___ pve-devel mailing list pve-devel@lists.

[pve-devel] [PATCH manager 4/6] ui: dc/UserView: disable Password button based on realm type

2021-07-01 Thread Dominik Csapak
Signed-off-by: Dominik Csapak --- www/manager6/Utils.js | 5 + www/manager6/dc/UserView.js | 9 + 2 files changed, 14 insertions(+) diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js index e642e283..fbd2dfa8 100644 --- a/www/manager6/Utils.js +++ b/www/manager6/Utils.js

[pve-devel] [PATCH manager 1/6] ui: dc/AuthEditBase: only add tfa field on auth modes that support that

2021-07-01 Thread Dominik Csapak
for now that are all, but we will ad one where that is not possible Signed-off-by: Dominik Csapak --- www/manager6/Utils.js | 3 +++ www/manager6/dc/AuthEditBase.js | 14 +- 2 files changed, 12 insertions(+), 5 deletions(-) diff --git a/www/manager6/Utils.js b/www/manager

[pve-devel] [PATCH manager 6/6] ui: Workspace: use domains info to hide password/tfa items

2021-07-01 Thread Dominik Csapak
in the user menu we have to make an additional api call here, since it is the only place (currently) where we can get the realm type Signed-off-by: Dominik Csapak --- alternatively, we could give that info in the index or the ticket api call, but this has the advantage that we will load the doma

[pve-devel] [PATCH access-control/manager] add openid gui handling

2021-07-01 Thread Dominik Csapak
for realm add/edit, as well as disabling/hiding irrelevant buttons pve-access-control: Dominik Csapak (1): api: user: add realmtype to user list src/PVE/API2/User.pm | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) pve-manager: Dominik Csapak (6): ui: dc/AuthEditBase:

[pve-devel] [PATCH access-control 1/1] api: user: add realmtype to user list

2021-07-01 Thread Dominik Csapak
this makes it much easier to determine if a user can e.g. change a password or tfa, based on realm Signed-off-by: Dominik Csapak --- src/PVE/API2/User.pm | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/src/PVE/API2/User.pm b/src/PVE/API2/User.pm index 05de57f.

[pve-devel] [PATCH manager 5/6] ui: Utils: refactor userid parsing to Utils

2021-07-01 Thread Dominik Csapak
we'll need that again Signed-off-by: Dominik Csapak --- www/manager6/Utils.js | 26 +- www/manager6/dc/UserView.js | 4 ++-- 2 files changed, 27 insertions(+), 3 deletions(-) diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js index fbd2dfa8..d807a37e 10064

[pve-devel] [PATCH manager 3/6] ui: dc/UserView: disable TFA button based on realm type

2021-07-01 Thread Dominik Csapak
Signed-off-by: Dominik Csapak --- www/manager6/dc/UserView.js | 9 + 1 file changed, 9 insertions(+) diff --git a/www/manager6/dc/UserView.js b/www/manager6/dc/UserView.js index 244c30f6..20ae3090 100644 --- a/www/manager6/dc/UserView.js +++ b/www/manager6/dc/UserView.js @@ -72,6 +72,15

[pve-devel] [PATCH manager 2/6] ui: dc: add AuthEditOpenId panel

2021-07-01 Thread Dominik Csapak
for adding and editing openid realms, does not support tfa Signed-off-by: Dominik Csapak --- www/manager6/Makefile | 1 + www/manager6/Utils.js | 6 +++ www/manager6/dc/AuthEditOpenId.js | 77 +++ 3 files changed, 84 insertions(+) create mod

[pve-devel] [PATCH ja.po 1/1] Signed-off-by: ribbon

2021-07-01 Thread ribbon
From: ribbon update ja.po for proxmox-ve 7.0 --- ja.po | 895 ++ 1 file changed, 335 insertions(+), 560 deletions(-) diff --git a/ja.po b/ja.po index 9cac23e..6d4dfd1 100644 --- a/ja.po +++ b/ja.po @@ -35,7 +35,7 @@ msgstr "/some/path"

[pve-devel] applied: [PATCH pve-access-control v2 0/5] add OpenId realms

2021-07-01 Thread Thomas Lamprecht
On 30.06.21 08:10, Dietmar Maurer wrote: > Changes in v2: > - also check if user is expired (in check_user_enabled) > - always die with newline > - rename "user-attr" to "username-claim" > > > Dietmar Maurer (5): > check_user_enabled: also check if user is expired > add OpenId configuration >

[pve-devel] [PATCH ja.po 0/1] ja.po patch for proxmox-ve 7.0

2021-07-01 Thread ribbon
*** BLURB HERE *** ribbon (1): Signed-off-by: ribbon ja.po | 895 ++ 1 file changed, 335 insertions(+), 560 deletions(-) -- 2.26.2 ___ pve-devel mailing list pve-devel@lists.proxmox.com htt

[pve-devel] [PATCH docs v2] pct: clarify needed systemd-versions for cgroupv2 support

2021-07-01 Thread Stoiko Ivanov
versions determined through systemd's changelog, and a few empiric tests. Signed-off-by: Stoiko Ivanov --- changes v1->v2: * fixed the wrong ubuntu-version in the note (I tested 16.04 LTS not 16.10) Huge thanks to Thomas for catching this!! pct.adoc | 13 +++-- 1 file changed, 11 inserti

Re: [pve-devel] applied: [PATCH qemu-server] live-restore: preload efidisk before starting VM

2021-07-01 Thread Stefan Reiter
On 7/1/21 12:57 PM, Thomas Lamprecht wrote: On 01.07.21 11:37, Stefan Reiter wrote: The efidisk never got restored correctly before, since we don't use the generic print_drive_commandline_full for it, and as such it didn't get a backing image attached. This not only causes the efidisk data to be

[pve-devel] applied: [PATCH qemu-server] live-restore: preload efidisk before starting VM

2021-07-01 Thread Thomas Lamprecht
On 01.07.21 11:37, Stefan Reiter wrote: > The efidisk never got restored correctly before, since we don't use the > generic print_drive_commandline_full for it, and as such it didn't get a > backing image attached. This not only causes the efidisk data to be lost > on restore, but also an error at

[pve-devel] [PATCH qemu-server] live-restore: preload efidisk before starting VM

2021-07-01 Thread Stefan Reiter
The efidisk never got restored correctly before, since we don't use the generic print_drive_commandline_full for it, and as such it didn't get a backing image attached. This not only causes the efidisk data to be lost on restore, but also an error at the end, since we try to remove a non-existing P

[pve-devel] [PATCH docs] pct: clarify needed systemd-versions for cgroupv2 support

2021-07-01 Thread Stoiko Ivanov
versions determined through systemd's changelog, and a few empiric tests. Signed-off-by: Stoiko Ivanov --- pct.adoc | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/pct.adoc b/pct.adoc index b03b6ba..42826bd 100644 --- a/pct.adoc +++ b/pct.adoc @@ -518,8 +518,17

[pve-devel] [PATCH pve-rs] apt: check if repository is already configured before adding

2021-07-01 Thread Fabian Ebner
and if it is, enable it. Signed-off-by: Fabian Ebner --- src/apt/repositories.rs | 25 ++--- 1 file changed, 22 insertions(+), 3 deletions(-) diff --git a/src/apt/repositories.rs b/src/apt/repositories.rs index 26af67c..79c2334 100644 --- a/src/apt/repositories.rs +++ b/src/

[pve-devel] [PATCH pve-docs] add OpenId Connect docu

2021-07-01 Thread Dietmar Maurer
--- pveum.adoc | 88 +- 1 file changed, 87 insertions(+), 1 deletion(-) diff --git a/pveum.adoc b/pveum.adoc index a1adbaa..9329583 100644 --- a/pveum.adoc +++ b/pveum.adoc @@ -29,7 +29,7 @@ endif::manvolnum[] Proxmox VE supports multiple aut

[pve-devel] [PATCH v11 manager 0/3] fix#1710: add download from url button

2021-07-01 Thread Lorenz Stechauner
changes to v10: * dropped already applied patches * added "check" button - the gui now does not automatically send the metadata request anymore * removed (visible) content type selector, because there was only one hard-coded option every time * added loading mask while the metadata check is in pr

[pve-devel] [PATCH v11 manager 1/3] api: nodes: add query_url_metadata method

2021-07-01 Thread Lorenz Stechauner
metadata is gained using a HEAD request. Due to the ability of this api endpoint to request files on internal networks (which would not be visible/accessible from outside) it is restricted to users with permissions `Sys.Audit` and `Sys.Modify` on `/`. Users with these permissions are able to alter

[pve-devel] [PATCH v11 manager 3/3] fix #1710: ui: storage: add download from url button

2021-07-01 Thread Lorenz Stechauner
uses the common function PVE::Tools::download_file_from_url to download a iso image or container template. note: Only users with permissions `Sys.Audit` and `Sys.Modify` on `/` are permitted to use the api endpoints due to security reasons. (it is possible to download files from internal networks

[pve-devel] [PATCH v11 manager 2/3] ui: Utils: change download task format

2021-07-01 Thread Lorenz Stechauner
Signed-off-by: Lorenz Stechauner --- www/manager6/Utils.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js index 3fdf37d3..c78467e7 100644 --- a/www/manager6/Utils.js +++ b/www/manager6/Utils.js @@ -1782,7 +1782,7 @@ Ext.define('PV

[pve-devel] [PATCH manager 2/2] pve6to7: drop PASS for Luminous -> Nautilus check

2021-07-01 Thread Fabian Grünbichler
this should pass on pretty much every system running hyper converged Ceph on PVE 6.x .. Signed-off-by: Fabian Grünbichler --- Notes: there are for sure more things to adapt w.r.t. Ceph.. PVE/CLI/pve6to7.pm | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/PVE/CLI/pve6t

[pve-devel] [PATCH manager 1/2] pve6to7: reduce number of cluster PASS statements

2021-07-01 Thread Fabian Grünbichler
these were mostly releveant for upgrading from Corosync 2.x to 3.x - so keep the warnings/errors, but reduce the noise a bit by skipping lots of PASS output. Signed-off-by: Fabian Grünbichler --- Notes: on a 3-node cluster this reduced the amount of output from 47 statements to 40, tota

[pve-devel] [PATCH pve-rs 1/1] apt: avoid overwriting files that could not be parsed

2021-07-01 Thread Fabian Ebner
The enable/disable case should not be reachable via UI as the digest check comes first, but it's still good to fix it too. Reported-by: Dominik Csapak Signed-off-by: Fabian Ebner --- src/apt/repositories.rs | 16 ++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/sr

[pve-devel] [PATCH proxmox-widget-toolkit 1/1] apt repos: use correct URL

2021-07-01 Thread Fabian Ebner
so there is a 'result' property, which the window expects. Suggested-by: Thomas Lamprecht Signed-off-by: Fabian Ebner --- src/node/APTRepositories.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/node/APTRepositories.js b/src/node/APTRepositories.js index baca834..53ff

[pve-devel] [PATCH-SERIES proxmox-widget-toolkit/pve-rs] Two more APT repo fixes

2021-07-01 Thread Fabian Ebner
proxmox-widget-toolkit: Fabian Ebner (1): apt repos: use correct URL src/node/APTRepositories.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) pve-rs: Fabian Ebner (1): apt: avoid overwriting files that could not be parsed src/apt/repositories.rs | 16 ++-- 1 file ch

[pve-devel] applied: [PATCH qemu-server] cfg2cmd/drive: don't use io_uring for krbd with wb/wt cache

2021-07-01 Thread Thomas Lamprecht
On 30.06.21 17:18, Stefan Reiter wrote: > As reported here and locally reproduced: > https://forum.proxmox.com/threads/efi-vms-wont-start-under-7-beta-with-writeback-cache.91629/ > > This configuration is currently broken. Until we figure out how to fix > it properly, we can just have this (luckil