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@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@redhat.com>