[pbs-devel] applied: [PATCH proxmox-backup v3 1/7] api2/tape/restore: return backup manifest in try_restore_snapshot_archive

Dietmar Maurer dietmar at proxmox.com
Fri May 7 12:49:57 CEST 2021


applied

On 5/6/21 2:20 PM, Dominik Csapak wrote:
> we'll use that for partial snapshot restore
>
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>   src/api2/tape/restore.rs | 11 ++++++-----
>   1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/src/api2/tape/restore.rs b/src/api2/tape/restore.rs
> index f3452364..9884b379 100644
> --- a/src/api2/tape/restore.rs
> +++ b/src/api2/tape/restore.rs
> @@ -800,7 +800,7 @@ fn try_restore_snapshot_archive<R: pxar::decoder::SeqRead>(
>       worker: Arc<WorkerTask>,
>       decoder: &mut pxar::decoder::sync::Decoder<R>,
>       snapshot_path: &Path,
> -) -> Result<(), Error> {
> +) -> Result<BackupManifest, Error> {
>   
>       let _root = match decoder.next() {
>           None => bail!("missing root entry"),
> @@ -886,9 +886,10 @@ fn try_restore_snapshot_archive<R: pxar::decoder::SeqRead>(
>           }
>       }
>   
> -    if manifest.is_none() {
> -        bail!("missing manifest");
> -    }
> +    let manifest = match manifest {
> +        None => bail!("missing manifest"),
> +        Some(manifest) => manifest,
> +    };
>   
>       // Do not verify anything here, because this would be to slow (causes tape stops).
>   
> @@ -902,7 +903,7 @@ fn try_restore_snapshot_archive<R: pxar::decoder::SeqRead>(
>           bail!("Atomic rename manifest {:?} failed - {}", manifest_path, err);
>       }
>   
> -    Ok(())
> +    Ok(manifest)
>   }
>   
>   /// Try to restore media catalogs (form catalog_archives)





More information about the pbs-devel mailing list