[pbs-devel] [PATCH proxmox-backup 4/6] fix #2865: detect and skip vanished snapshots
Fabian Grünbichler
f.gruenbichler at proxmox.com
Tue Jul 21 15:03:35 CEST 2020
also when they have been removed/forgotten since we retrieved the
snapshot list for the currently syncing backup group.
Signed-off-by: Fabian Grünbichler <f.gruenbichler at proxmox.com>
---
Notes:
there is probably a more elegant ways to do this?
src/client/pull.rs | 24 +++++++++++++++++++++++-
1 file changed, 23 insertions(+), 1 deletion(-)
diff --git a/src/client/pull.rs b/src/client/pull.rs
index bbe01969..421260a3 100644
--- a/src/client/pull.rs
+++ b/src/client/pull.rs
@@ -7,6 +7,7 @@ use std::sync::Arc;
use std::collections::HashMap;
use std::io::{Seek, SeekFrom};
+use proxmox::api::error::{StatusCode, HttpError};
use crate::server::{WorkerTask};
use crate::backup::*;
use crate::api2::types::*;
@@ -151,7 +152,28 @@ async fn pull_snapshot(
let mut tmp_manifest_name = manifest_name.clone();
tmp_manifest_name.set_extension("tmp");
- let mut tmp_manifest_file = download_manifest(&reader, &tmp_manifest_name).await?;
+ let download_res = download_manifest(&reader, &tmp_manifest_name).await;
+ let mut tmp_manifest_file = match download_res {
+ Ok(manifest_file) => manifest_file,
+ Err(err) => {
+ match err.downcast_ref::<HttpError>() {
+ Some(HttpError { code, message }) => {
+ match code {
+ &StatusCode::NOT_FOUND => {
+ worker.log(format!("skipping snapshot {} - vanished since start of sync", snapshot));
+ return Ok(());
+ },
+ _ => {
+ bail!("HTTP error {} - {}", code, message);
+ },
+ }
+ },
+ None => {
+ return Err(err);
+ },
+ };
+ },
+ };
let tmp_manifest_blob = DataBlob::load(&mut tmp_manifest_file)?;
tmp_manifest_blob.verify_crc()?;
--
2.20.1
More information about the pbs-devel
mailing list