[pbs-devel] applied-series: [PATCH v7 proxmox-backup 00/31] fix #3044: push datastore to remote target

Christian Ebner c.ebner at proxmox.com
Thu Nov 21 13:26:40 CET 2024


On 11/21/24 13:08, Fabian Grünbichler wrote:
> with some follow-ups, notably:
> 
> - dropped the named features, checking just the api version instead
> - simplified the namespace filtering
> - added a new datastore_api_path helper
> - add a check whether the source namespace anchor exists
> - simplify the snapshot selection for remove_vanished
> - reduce the (expensive) queries to the remote snapshot lists
> - some code simplification and style cleanups
> 
> the error handling/contexts could still benefit from some attention
> (mostly making clear where an error from the remote side is bubbled up,
> since the ACL paths are confusing if that context is not included), but
> that can be done as follow-up..

Acked, thanks a lot for all your input on this series!

Will look at the suggested improved error handling right away.




More information about the pbs-devel mailing list