[pbs-devel] [PATCH proxmox{, -backup} v4 00/48] fix #2943: S3 storage backend for datastores

[Christian Ebner]

On 7/1/25 16:30, Hannes Duerr wrote:
> Hi i am currently testing this series and noticed that you've missed the 
> following build dependencies in the debian/control file for the proxmox- 
> backup repository:
> librust-iso8601-dev,
> librust-md5-dev,
> librust-quick-xml-dev,
> librust-serde-xml-rs-dev,

Acked, will be included in the upcoming version of the patches.

To keep everybody in the loop, Lukas already reported some issues 
regarding this series off-list:

The `s3-refresh` command is still not working correctly, failing if some 
type directories (vm, ct, host, ns) are missing/not present yet. Also 
the cli command does not work in the current iteration of the patches 
since it requires matching the handler method by `ApiHandler::Async`.

Further, further testing by me showed that there was an issue leading to 
possibly missing chunks in case the upload to the S3 object store 
failed, but the chunk being inserted to the local cache store 
nevertheless (leading to other clients skipping upload).

Also, logging and error context is still being improved upon.

Above issues will be addressed with the next iteration of the patches.

Thanks for testing!