]> git.proxmox.com Git - pve-guest-common.git/commit
fix #3111: config: snapshot delete: check if replication still needs it
authorFabian Ebner <f.ebner@proxmox.com>
Thu, 12 Aug 2021 11:01:11 +0000 (13:01 +0200)
committerFabian Grünbichler <f.gruenbichler@proxmox.com>
Mon, 8 Nov 2021 09:34:14 +0000 (10:34 +0100)
commit602ca77cdb216e0ec5ba268b2ad42b0eee6f7f66
treeb1ce387cdb971ae6ab032ebb8c724b35b6e1e086
parent8d1cd443459d95431a3235d0bf18ec1de491e87a
fix #3111: config: snapshot delete: check if replication still needs it

and abort if it does and --force is not specified.

After rollback, the rollback snapshot might still be needed as the
base for incremental replication, because rollback removes (blocking)
replication snapshots.

It's not enough to limit the check to the most recent snapshot,
because new snapshots might've been created between rollback and
remove.

It's not enough to limit the check to snapshots without a parent (i.e.
in case of ZFS, the oldest), because some volumes might've been added
only after that, meaning the oldest snapshot is not an incremental
replication base for them.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>
src/PVE/AbstractConfig.pm
src/PVE/Replication.pm
src/PVE/ReplicationConfig.pm