[pve-devel] [PATCH-SERIES] correctly handle warnings status
Fabian Ebner
f.ebner at proxmox.com
Wed May 12 14:32:49 CEST 2021
There's a few places where we use the logic "OK" if and only if no error for a
task status, but this is no longer true now that we also support ending in a
warnings status (not yet used by anything, but the feature is there).
Everything depends on the first patch and depedency bumps for pve-common
are needed across the board.
pve-common:
Fabian Ebner (1):
tools: add upid_status_is_error function
src/PVE/Tools.pm | 8 ++++++++
1 file changed, 8 insertions(+)
pve-manager:
Fabian Ebner (4):
api/cli: tasks: don't treat warnings status as an error status
nodes: startall: correctly handle warning status for delayed task
cli: pveam: remove unused private sub
cli tools: correctly handle warnings status
PVE/API2/Nodes.pm | 2 +-
PVE/API2/Tasks.pm | 5 ++++-
PVE/CLI/pveam.pm | 6 ------
PVE/CLI/pveceph.pm | 2 +-
PVE/CLI/pvenode.pm | 7 +++++--
PVE/CLI/vzdump.pm | 2 +-
6 files changed, 12 insertions(+), 12 deletions(-)
pve-storage:
Fabian Ebner (1):
api: content: correctly handle warnings status for delayed task
PVE/API2/Storage/Content.pm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
pve-container:
Fabian Ebner (1):
pct: correctly handle warnings task status
src/PVE/CLI/pct.pm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
qemu-server:
Fabian Ebner (2):
cli tools: correctly handle warnings task status
api: update vm: correctly handle warnings status for delayed task
PVE/API2/Qemu.pm | 2 +-
PVE/CLI/qm.pm | 2 +-
PVE/CLI/qmrestore.pm | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
--
2.20.1
More information about the pve-devel
mailing list