Re: [pbs-devel] [PATCH proxmox-backup] verify: include namespace for group in log

["Michael Köppl" <m.koeppl@proxmox.com>]

Quickly tested this by verifying both a group in my root namespace as
well as a custom namespace. Works as expected and seems like a sensible
addition to the logs.

Consider this:
Tested-by: Michael Köppl <m.koeppl@proxmox.com>

On Thu Nov 20, 2025 at 12:52 PM CET, Hannes Laimer wrote:
> Groups with the same name in different namespaces couldn't really be told
> appart. This solves this by also printing the namespace of a group to the

s/appart/apart

> log.
>
> This came up in support.
>
> Signed-off-by: Hannes Laimer <h.laimer@proxmox.com>
> ---
> not super sure about the formatting, didn't want to change the
> <datastore>:<group> format we use a lot. don't log for snapshots as that
> would potentially add a lot of output that doesn't add much information
>
>  src/backup/verify.rs | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/src/backup/verify.rs b/src/backup/verify.rs
> index 734c7b30..10f54c28 100644
> --- a/src/backup/verify.rs
> +++ b/src/backup/verify.rs
> @@ -500,8 +500,15 @@ impl VerifyWorker {
>          };
>  
>          let snapshot_count = list.len();
> +        let ns = group.backup_ns();
> +        let ns_str = if ns.is_root() {
> +            "/".to_string()
> +        } else {
> +            ns.to_string()
> +        };
>          info!(
> -            "verify group {}:{} ({} snapshots)",
> +            "verify group in '{}' - {}:{} ({} snapshots)",
> +            ns_str,
>              self.datastore.name(),
>              group.group(),
>              snapshot_count



_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel