Re: [pve-devel] [PATCH cd-builder 2/2] fix #1999: create a tree view for qm listsnapshot

2019-05-10 Thread Rhonda D'Vine
Hi, thanks for the suggestions, they helped a lot. :) On 5/8/19 11:26 AM, Thomas Lamprecht wrote: > On 5/3/19 11:48 AM, Rhonda D'Vine wrote: >> @@ -922,12 +922,67 @@ our $cmddef = { >> listsnapshot => [ "PVE::API2::Qemu", 'snapshot_list', ['vmid'], { node >> => $nodename }, >>

Re: [pve-devel] [PATCH cd-builder 2/2] fix #1999: create a tree view for qm listsnapshot

2019-05-08 Thread Thomas Lamprecht
On 5/3/19 11:48 AM, Rhonda D'Vine wrote: > The look of the tree is based on the GUI variant, so that we have a > consistent output when run multiple times, too. > in principle ok, comments to implementation details below, thanks! > Signed-off-by: Rhonda D'Vine > --- > PVE/CLI/qm.pm | 63 > +++

[pve-devel] [PATCH cd-builder 2/2] fix #1999: create a tree view for qm listsnapshot

2019-05-03 Thread Rhonda D'Vine
The look of the tree is based on the GUI variant, so that we have a consistent output when run multiple times, too. Signed-off-by: Rhonda D'Vine --- PVE/CLI/qm.pm | 63 +++ 1 file changed, 59 insertions(+), 4 deletions(-) diff --git a/PVE/