[pbs-devel] applied: [PATCH v3 proxmox-backup] partial fix #5560: client: periodically show backup progress

[Thomas Lamprecht]

Am 18/10/2024 um 10:33 schrieb Christian Ebner:
> On 10/18/24 09:53, Thomas Lamprecht wrote:
>> Minus the s/opt-in/opt-out/ preference that's fine by me. We could then
>> also switch to a higher reporting rate in PVE, albeit FWIW, for VM backups
>> there we use 10s and then slow down after some time, trying to take a
>> balance for early fast feedback and short backups and long backups.
>> And just throwing out ideas, one could also do a size-related interval,
>> like print a report every (at least) X MB or GB uploaded.
> 
> Hmm, that would be nice to have, maybe allow to pass either a `TimeSpan` 
> or a `HumanByte` for the log interval parameter? Not sure if parsing 
> would work without possible value clashes though (e.g. 4M could be 4 
> months or 4 MiB...).
> 
> But maybe that is overkill and not so user friendly, the better option 
> being to set either time or size based intervals by (yet another) flag.
> 

Yeah, if we do this I'd probably separate it by type, as while it might
work out for this specific one mixing units can get confusing fast in
general.