Re: [pve-devel] [PATCH storage] Fix #582: Make removing backups a background task

2019-05-07 Thread Dominik Csapak
i would prefer that it behaves like the update_vm_api sub in PVE/API2/Qemu.pm (line ~1000) there we have a parameter 'background_delay' which we could use to make the removal async (this can already work in the gui) and the return value is type string but optional, so you could return undef witho

[pve-devel] [PATCH storage] Fix #582: Make removing backups a background task

2019-05-02 Thread Dominic Jäger
Previously, the web gui timed out when removing a backup took long. Doing the main part of the API DELETE call in a fork_worker solves this. Signed-off-by: Dominic Jäger --- Thomas suggested returning undef on bugzilla. Returning "" is what seemed closest to this while still being of return type