[pve-devel] applied-series: [PATCH-SERIES v4 container/qemu-server] fix #3424: wait for active replication when deleting a snapshot
Fabian Grünbichler
f.gruenbichler at proxmox.com
Tue Mar 15 13:34:45 CET 2022
with some small follow-ups as discussed off-list:
- improve error messages in case of failure to acquire lock
- downgrade die to warn in pve-container vzdump patch, since it's not
fatal
On February 23, 2022 1:03 pm, Fabian Ebner wrote:
> Avoid that an attempt to remove a snapshot that's actively used by
> replication leads to a partially (or not) removed snapshot and locked
> guest.
>
> I decided to make the checks at the call sides, because passing the
> log function and timeout to snapshot_delete felt awkward as they
> would only be used for obtaining the lock.
>
> Changes from v3:
> * Unconditionally take the lock, to not race with replication job
> creation and future-proofing.
> * Only log in the case with the long timeout if we can't obtain
> the lock quickly.
> * Make message more general, because it might be another snapshot
> removal operation holding the lock.
>
>
> container:
>
> Fabian Ebner (2):
> partially fix #3424: vzdump: cleanup: wait for active replication
> fix #3424: api: snapshot delete: wait for active replication
>
> src/PVE/API2/LXC/Snapshot.pm | 11 ++++++++++-
> src/PVE/VZDump/LXC.pm | 20 ++++++++++++++++++--
> 2 files changed, 28 insertions(+), 3 deletions(-)
>
>
> qemu-server:
>
> Fabian Ebner (1):
> fix #3424: api: snapshot delete: wait for active replication
>
> PVE/API2/Qemu.pm | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
>
> --
> 2.30.2
>
>
>
> _______________________________________________
> pve-devel mailing list
> pve-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
>
>
>
More information about the pve-devel
mailing list