On Thu, Dec 08, 2022 at 14:30:51 +0100, Pavel Hrdina wrote:
This simplifies the code a bit by reusing existing parts that
deletes
a single snapshot.
The drawback of this change is that we will now call the re-parent bits
to keep the metadata in sync for every child even though it will get
deleted as well.
Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com>
---
src/qemu/qemu_snapshot.c | 69 +++++++++++++++++++++-------------------
1 file changed, 36 insertions(+), 33 deletions(-)
diff --git a/src/qemu/qemu_snapshot.c b/src/qemu/qemu_snapshot.c
index 3d5467457b..846f6e9e53 100644
--- a/src/qemu/qemu_snapshot.c
+++ b/src/qemu/qemu_snapshot.c
@@ -2311,6 +2311,31 @@ qemuSnapshotDeleteSingle(virDomainObj *vm,
}
+struct qemuSnapshotDeleteAllData {
+ virDomainObj *vm;
+ bool metadata_only;
+ int error;
+};
+
+
+static int
+qemuSnapshotDeleteAll(void *payload,
Since this itself doesn't delete all snapshots please call it e.g:
qemuSnapshotDeleteAllHelper
Reviewed-by: Peter Krempa <pkrempa(a)redhat.com>