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

Fabian Ebner f.ebner at proxmox.com
Fri Apr 9 10:44:29 CEST 2021

Changes from v1:
    * dropped already applied patches
    * add format_task_status helper and use that everywhere
    * improve usage of replace() (for the widget-toolkit change)
    * chomp the message in warn() (for the pve-common change)

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 proxmox-widget-toolkit
pve-container needs a dependency bump for pve-common


Fabian Ebner (3):
  task status: create helper for formatting
  format task status: rename variable
  format task status: improve replace() usage

 src/Utils.js      | 11 +++++++++++
 src/node/Tasks.js | 10 +---------
 2 files changed, 12 insertions(+), 9 deletions(-)


Fabian Ebner (1):
  ui: tasks: use format_task_status

 www/panel/Tasks.js | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)


Fabian Ebner (1):
  ui: cluster task log: handle warnings like the node task log does

 www/manager6/dc/Tasks.js | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)


Fabian Ebner (1):
  allow workers to count warnings and finish tasks in a WARNINGS state

 src/PVE/RESTEnvironment.pm | 22 ++++++++++++++++++++--
 src/PVE/Tools.pm           |  2 ++
 2 files changed, 22 insertions(+), 2 deletions(-)


Fabian Ebner (1):
  restore: sanitize config: use new warn() function

 src/PVE/LXC/Create.pm | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)


More information about the pve-devel mailing list