* [pbs-devel] [PATCH proxmox-backup] fix #4357: datastore: take into account namespaces when marking chunks
@ 2023-04-20 8:08 Stefan Sterz
2023-05-17 9:35 ` [pbs-devel] applied: " Thomas Lamprecht
0 siblings, 1 reply; 2+ messages in thread
From: Stefan Sterz @ 2023-04-20 8:08 UTC (permalink / raw)
To: pbs-devel
previously when marking used chunks the namespace wasn't taken into
account and valid snapshots were marked as "strange paths". this lead
to a line in the log of a gc job such as this:
found (and marked) 2 index files outside of expected directory scheme
which some users perceived as an error. parse the namespace too and
only mark the path as strange if parsing the namespace and/or backup
dir fails.
Signed-off-by: Stefan Sterz <s.sterz@proxmox.com>
---
pbs-datastore/src/datastore.rs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/pbs-datastore/src/datastore.rs b/pbs-datastore/src/datastore.rs
index bf951305..a500c0ab 100644
--- a/pbs-datastore/src/datastore.rs
+++ b/pbs-datastore/src/datastore.rs
@@ -964,7 +964,7 @@ impl DataStore {
if let Some(backup_dir_path) = img.parent() {
let backup_dir_path = backup_dir_path.strip_prefix(self.base_path())?;
if let Some(backup_dir_str) = backup_dir_path.to_str() {
- if pbs_api_types::BackupDir::from_str(backup_dir_str).is_err() {
+ if pbs_api_types::parse_ns_and_snapshot(backup_dir_str).is_err() {
strange_paths_count += 1;
}
}
--
2.30.2
^ permalink raw reply [flat|nested] 2+ messages in thread
* [pbs-devel] applied: [PATCH proxmox-backup] fix #4357: datastore: take into account namespaces when marking chunks
2023-04-20 8:08 [pbs-devel] [PATCH proxmox-backup] fix #4357: datastore: take into account namespaces when marking chunks Stefan Sterz
@ 2023-05-17 9:35 ` Thomas Lamprecht
0 siblings, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2023-05-17 9:35 UTC (permalink / raw)
To: Proxmox Backup Server development discussion, Stefan Sterz
Am 20/04/2023 um 10:08 schrieb Stefan Sterz:
> previously when marking used chunks the namespace wasn't taken into
> account and valid snapshots were marked as "strange paths". this lead
> to a line in the log of a gc job such as this:
>
> found (and marked) 2 index files outside of expected directory scheme
>
> which some users perceived as an error. parse the namespace too and
> only mark the path as strange if parsing the namespace and/or backup
> dir fails.
>
> Signed-off-by: Stefan Sterz <s.sterz@proxmox.com>
> ---
> pbs-datastore/src/datastore.rs | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>
applied, thanks!
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-05-17 9:35 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-20 8:08 [pbs-devel] [PATCH proxmox-backup] fix #4357: datastore: take into account namespaces when marking chunks Stefan Sterz
2023-05-17 9:35 ` [pbs-devel] applied: " Thomas Lamprecht
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox