[pve-devel] applied: [PATCH storage 2/3] convert maxfiles to prune_backups when reading the storage configuration
Thomas Lamprecht
t.lamprecht at proxmox.com
Mon Nov 23 15:58:07 CET 2020
On 23.11.20 13:33, Fabian Ebner wrote:
> If there are already prune options configured, simply delete the maxfiles
> setting. Having set both is invalid from vzdump's perspective anyways, and any
> backup job on such a storage failed, meaning a user would've noticed.
>
> If there are no prune options, translate the maxfiles value to keep-last,
> except for maxfiles being zero (=unlimited), in which case we use keep-all.
>
> If both are not set, don't set anything, so:
> 1. Storages don't suddenly have retention options set.
> 2. People relying on vzdump defaults can still use those.
>
> Signed-off-by: Fabian Ebner <f.ebner at proxmox.com>
> ---
> PVE/Storage.pm | 22 ++++++++++++++++++++++
> 1 file changed, 22 insertions(+)
>
>
applied, thanks!
More information about the pve-devel
mailing list