[pve-devel] [PATCH storage 2/2] tests: zfs: complain when a first sub-test dies

2021-03-12 Thread Fabian Ebner
Previously, $count was not increased and no test failure was reported. Signed-off-by: Fabian Ebner --- Found by messing up in the previous patch at first with: my $res = map { $_ => [] } keys %{$ids}; instead of my $res = { map { $_ => [] } keys %{$ids} }; and wondering why it failed in

[pve-devel] [PATCH storage 1/2] vdisk list: only collect images from storages with an appropriate content type

2021-03-12 Thread Fabian Ebner
Only these storages are activated in the first place, and it's bad behavior to list images when no appropriate content type is not set. For example, on VM destruction, this avoids unreferenced images to be deleted from a storage with only 'backup' content type set, which is supposedly what happene

[pve-devel] [PATCH widget-toolkit 1/6] tasks: add warningsText

2021-03-12 Thread Fabian Ebner
to avoid having capitalized 'WARNINGS' (especially since 'Error' is not) and support different languages. Signed-off-by: Fabian Ebner --- src/Utils.js | 1 + src/node/Tasks.js | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/Utils.js b/src/Utils.js index af5f1db..2

[pve-devel] [PATCH manager 3/6] ui: cluster task log: eslint fixes

2021-03-12 Thread Fabian Ebner
Signed-off-by: Fabian Ebner --- www/manager6/dc/Tasks.js | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/www/manager6/dc/Tasks.js b/www/manager6/dc/Tasks.js index c244bd2e..c5075dc3 100644 --- a/www/manager6/dc/Tasks.js +++ b/www/manager6/dc/Tasks.js @@ -82,9 +82,9 @@

[pve-devel] [PATCH-SERIES] improve warnings handling in UI and add to PVE

2021-03-12 Thread Fabian Ebner
The first half is just a small UI improvement (but it's all over the place), replacing 'WARNINGS' with (a language-aware) 'Warnings'. The later half adds the feature of ending in a WARNINGS state to PVE and provides a usage example. proxmox-backup and pve-manager need a dependency bump for proxm

[pve-devel] [PATCH/RFC common 5/6] allow workers to count warnings and finish tasks in a WARNINGS state

2021-03-12 Thread Fabian Ebner
as is already supported by the UI (and PBS). A nice bonus is that warn() can be used by both workers and non-workers, while for workers the output is redirected/duplicated as set up by {fork,tee}_worker() and non-erroring workers that issued a warning will end in a WARNINGS state. Suggested-by: T

[pve-devel] [PATCH/RFC container 6/6] restore: sanitize config: use new warn() function

2021-03-12 Thread Fabian Ebner
to make it more visible when the task finished with warnings. Signed-off-by: Fabian Ebner --- Dependency bump for pve-common is needed. src/PVE/LXC/Create.pm | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/src/PVE/LXC/Create.pm b/src/PVE/LXC/Create.pm index

[pve-devel] [PATCH manager 4/6] ui: cluster task log: handle warnings like the node task log does

2021-03-12 Thread Fabian Ebner
Copied the relevant code from widget-toolkit. Signed-off-by: Fabian Ebner --- Dependency bump for widget-toolkit is needed. www/manager6/dc/Tasks.js | 23 +-- 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/www/manager6/dc/Tasks.js b/www/manager6/dc/Tasks.js

[pve-devel] [PATCH proxmox-backup 2/6] ui: tasks: use warningsText

2021-03-12 Thread Fabian Ebner
to avoid having capitalized 'WARNINGS' (especially since 'Error' is not) and support different languages. Signed-off-by: Fabian Ebner --- Dependency bump for widget-toolkit is needed. www/panel/Tasks.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/www/panel/Tasks.js b/

[pve-devel] [PATCH zfsonlinux] update submodule and patches to zfs-2.0.4

2021-03-12 Thread Stoiko Ivanov
Signed-off-by: Stoiko Ivanov --- 2.0.4 was tagged earlier this week and contains a few minor fixes (including Thomas' patch for zpool output). Tested very roughly by running ztest on a VM ...-intend-continuation-from-removal-st.patch | 48 --- debian/patches/series