Re: [pve-devel] [PATCH qemu-server] api: snapshot create/delete/rollback: print log line that mentions snapshot name

2024-05-29 Thread Fabian Grünbichler
On May 29, 2024 1:07 pm, Dominik Csapak wrote: > On 5/29/24 12:09, Fabian Grünbichler wrote: >> On May 29, 2024 11:20 am, Dominik Csapak wrote: >>> so that an admin can see from the task logs which snapshot was rolled >>> back/created/removed without the need to look into the journal (to which >>>

Re: [pve-devel] [PATCH qemu-server] api: snapshot create/delete/rollback: print log line that mentions snapshot name

2024-05-29 Thread Dominik Csapak
On 5/29/24 12:09, Fabian Grünbichler wrote: On May 29, 2024 11:20 am, Dominik Csapak wrote: so that an admin can see from the task logs which snapshot was rolled back/created/removed without the need to look into the journal (to which the admin might not have access to). good idea in general,

Re: [pve-devel] [PATCH qemu-server] api: snapshot create/delete/rollback: print log line that mentions snapshot name

2024-05-29 Thread Fabian Grünbichler
On May 29, 2024 11:20 am, Dominik Csapak wrote: > so that an admin can see from the task logs which snapshot was rolled > back/created/removed without the need to look into the journal (to which > the admin might not have access to). good idea in general, but wouldn't it be better to log it

[pve-devel] [PATCH qemu-server] api: snapshot create/delete/rollback: print log line that mentions snapshot name

2024-05-29 Thread Dominik Csapak
so that an admin can see from the task logs which snapshot was rolled back/created/removed without the need to look into the journal (to which the admin might not have access to). Signed-off-by: Dominik Csapak --- noticed while trying to find out, to which snapshot a vm was rolled back to on a