]> git.proxmox.com Git - pve-storage.git/commit
plugins: allow limiting the number of protected backups per guest
authorFabian Ebner <f.ebner@proxmox.com>
Tue, 29 Mar 2022 12:53:13 +0000 (14:53 +0200)
committerFabian Grünbichler <f.gruenbichler@proxmox.com>
Wed, 6 Apr 2022 07:47:12 +0000 (09:47 +0200)
commit8009417d0dac475183a06f56dc8a27c0e223eb48
tree036ee75a51af661a2d08872ffedf5a42b0184042
parent89a7507c7aed62fb7e565fe7d5c0bb013509a5e3
plugins: allow limiting the number of protected backups per guest

The ability to mark backups as protected broke the implicit assumption
in vzdump that remove=1 and current number of backups being the limit
(i.e. sum of all keep options) will result in a backup being removed.

Introduce a new storage property 'max-protected-backups' to limit the
number of protected backups per guest. Use 5 as a default value, as it
should cover most use cases, while still not having too big of a
potential overhead in many scenarios.

For external plugins that do not return the backup subtype in
list_volumes, all protected backups with the same ID will count
towards the limit.

An alternative would be to count the protected backups when pruning.
While that would avoid the need for a new property, it would break the
current semantics of protected backups being ignored for pruning. It
also would be less flexible, e.g. for PBS, it can make sense to have
both keep-all=1 and a limit for the number of protected snapshots on
the PVE side.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>
PVE/Storage.pm
PVE/Storage/BTRFSPlugin.pm
PVE/Storage/CIFSPlugin.pm
PVE/Storage/CephFSPlugin.pm
PVE/Storage/DirPlugin.pm
PVE/Storage/GlusterfsPlugin.pm
PVE/Storage/NFSPlugin.pm
PVE/Storage/PBSPlugin.pm
PVE/Storage/Plugin.pm