From 359191c83fd6c9d7f6b7d0f7e0dfcc6d2af4ab36 Mon Sep 17 00:00:00 2001 From: "Joshua M. Boniface" Date: Tue, 21 May 2024 16:38:47 -0400 Subject: [PATCH] Ensure snapshot name does not already exist --- daemon-common/vm.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/daemon-common/vm.py b/daemon-common/vm.py index 11090b5b..43527583 100644 --- a/daemon-common/vm.py +++ b/daemon-common/vm.py @@ -1265,6 +1265,12 @@ def create_vm_snapshot(zkhandler, domain, snapshot_name=None): False, f'ERROR: Snapshot name "{snapshot_name}" contains invalid characters; only alphanumeric, ".", "-", and "_" characters are allowed!', ) + current_snapshots = zkhandler.children(("domain.snapshots", dom_uuid)) + if current_snapshots and snapshot_name in current_snapshots: + return ( + False, + f'ERROR: Snapshot name "{snapshot_name}" already exists for VM "{domain}"!', + ) tstart = time.time()