[pbs-devel] applied: [PATCH proxmox-backup 2/2] api2: tape: restore: add warning for list restore

[Dietmar Maurer]

applied

On 7/13/21 11:11 AM, Dominik Csapak wrote:
> if an error occurs, the snapshot dirs will already be created, and we
> do not clean them up (some might already be finished).
>
> Warn the user that they are not cleaned up.
>
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>   src/api2/tape/restore.rs | 4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/src/api2/tape/restore.rs b/src/api2/tape/restore.rs
> index 68033c4a..f959a856 100644
> --- a/src/api2/tape/restore.rs
> +++ b/src/api2/tape/restore.rs
> @@ -647,6 +647,10 @@ fn restore_list_worker(
>           Ok(())
>       });
>   
> +    if res.is_err() {
> +        task_warn!(worker, "Error during restore, partially restored snapshots will NOT be cleaned up");
> +    }
> +
>       match std::fs::remove_dir_all(&base_path) {
>           Ok(()) => {}
>           Err(err) => task_warn!(worker, "error cleaning up: {}", err),