verification job: log failed dirs
else users have to manually search through a potentially very long task log to find the entries that are different.. this is the same summary printed at the end of a manual verify task. Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
This commit is contained in:
parent
81d10c3b37
commit
a4fa3fc241
|
@ -695,7 +695,7 @@ pub fn verify(
|
||||||
verify_all_backups(datastore, worker.clone(), worker.upid(), owner, None)?
|
verify_all_backups(datastore, worker.clone(), worker.upid(), owner, None)?
|
||||||
};
|
};
|
||||||
if failed_dirs.len() > 0 {
|
if failed_dirs.len() > 0 {
|
||||||
worker.log("Failed to verify following snapshots/groups:");
|
worker.log("Failed to verify the following snapshots/groups:");
|
||||||
for dir in failed_dirs {
|
for dir in failed_dirs {
|
||||||
worker.log(format!("\t{}", dir));
|
worker.log(format!("\t{}", dir));
|
||||||
}
|
}
|
||||||
|
|
|
@ -69,8 +69,15 @@ pub fn do_verification_job(
|
||||||
|
|
||||||
let result = verify_all_backups(datastore, worker.clone(), worker.upid(), None, Some(&filter));
|
let result = verify_all_backups(datastore, worker.clone(), worker.upid(), None, Some(&filter));
|
||||||
let job_result = match result {
|
let job_result = match result {
|
||||||
Ok(ref errors) if errors.is_empty() => Ok(()),
|
Ok(ref failed_dirs) if failed_dirs.is_empty() => Ok(()),
|
||||||
Ok(_) => Err(format_err!("verification failed - please check the log for details")),
|
Ok(ref failed_dirs) => {
|
||||||
|
worker.log("Failed to verify the following snapshots/groups:");
|
||||||
|
for dir in failed_dirs {
|
||||||
|
worker.log(format!("\t{}", dir));
|
||||||
|
}
|
||||||
|
|
||||||
|
Err(format_err!("verification failed - please check the log for details"))
|
||||||
|
},
|
||||||
Err(_) => Err(format_err!("verification failed - job aborted")),
|
Err(_) => Err(format_err!("verification failed - job aborted")),
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue