tyler/proxmox-backup
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

remove BackupGroup::list_groups

Browse Source 
BackupInfo::list_backup_groups is identical code-wise, and makes more
sense as entry point for listing groups.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
This commit is contained in:
Fabian Grünbichler
2020-11-30 16:27:18 +01:00
committed by Dietmar Maurer
parent 844660036b
commit 7f3b0f67e7
4 changed files with 4 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/backup/backup_info.rs
View File

@ -145,20 +145,6 @@ impl BackupGroup {
Ok(last)
}
pub fn list_groups(base_path: &Path) -> Result<Vec<BackupGroup>, Error> {
let mut list = Vec::new();
tools::scandir(libc::AT_FDCWD, base_path, &BACKUP_TYPE_REGEX, |l0_fd, backup_type, file_type| {
if file_type != nix::dir::Type::Directory { return Ok(()); }
tools::scandir(l0_fd, backup_type, &BACKUP_ID_REGEX, |_l1_fd, backup_id, file_type| {
if file_type != nix::dir::Type::Directory { return Ok(()); }
list.push(BackupGroup::new(backup_type, backup_id));
Ok(())
})
})?;
Ok(list)
}
}
impl std::fmt::Display for BackupGroup {

2
src/backup/verify.rs
View File

@ -533,7 +533,7 @@ pub fn verify_all_backups(
}
};
let mut list = match BackupGroup::list_groups(&datastore.base_path()) {
let mut list = match BackupInfo::list_backup_groups(&datastore.base_path()) {
Ok(list) => list
.into_iter()
.filter(|group| !(group.backup_type() == "host" && group.backup_id() == "benchmark"))