fix #2860: skip in-progress snapshots when syncing
they don't have a final manifest yet and are not done, so they can't be synced either. Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
This commit is contained in:
parent
13d9fe3a6c
commit
86f6f74114
|
@ -6,7 +6,6 @@ use std::convert::TryFrom;
|
||||||
use std::sync::Arc;
|
use std::sync::Arc;
|
||||||
use std::collections::HashMap;
|
use std::collections::HashMap;
|
||||||
use std::io::{Seek, SeekFrom};
|
use std::io::{Seek, SeekFrom};
|
||||||
use chrono::{Utc, TimeZone};
|
|
||||||
|
|
||||||
use crate::server::{WorkerTask};
|
use crate::server::{WorkerTask};
|
||||||
use crate::backup::*;
|
use crate::backup::*;
|
||||||
|
@ -302,7 +301,16 @@ pub async fn pull_group(
|
||||||
let mut remote_snapshots = std::collections::HashSet::new();
|
let mut remote_snapshots = std::collections::HashSet::new();
|
||||||
|
|
||||||
for item in list {
|
for item in list {
|
||||||
let backup_time = Utc.timestamp(item.backup_time, 0);
|
let snapshot = BackupDir::new(item.backup_type, item.backup_id, item.backup_time);
|
||||||
|
|
||||||
|
// in-progress backups can't be synced
|
||||||
|
if let None = item.size {
|
||||||
|
worker.log(format!("skipping snapshot {} - in-progress backup", snapshot));
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
let backup_time = snapshot.backup_time();
|
||||||
|
|
||||||
remote_snapshots.insert(backup_time);
|
remote_snapshots.insert(backup_time);
|
||||||
|
|
||||||
if let Some(last_sync_time) = last_sync {
|
if let Some(last_sync_time) = last_sync {
|
||||||
|
@ -319,14 +327,12 @@ pub async fn pull_group(
|
||||||
new_client,
|
new_client,
|
||||||
None,
|
None,
|
||||||
src_repo.store(),
|
src_repo.store(),
|
||||||
&item.backup_type,
|
snapshot.group().backup_type(),
|
||||||
&item.backup_id,
|
snapshot.group().backup_id(),
|
||||||
backup_time,
|
backup_time,
|
||||||
true,
|
true,
|
||||||
).await?;
|
).await?;
|
||||||
|
|
||||||
let snapshot = BackupDir::new(item.backup_type, item.backup_id, item.backup_time);
|
|
||||||
|
|
||||||
pull_snapshot_from(worker, reader, tgt_store.clone(), &snapshot).await?;
|
pull_snapshot_from(worker, reader, tgt_store.clone(), &snapshot).await?;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue