tools clean: also clean the snapshots

Signed-off-by: Gaëtan Lehmann <gaetan.lehmann@vates.tech>

Author: glehmann

lib/tools/cli.py

@@ -105,8 +105,8 @@ def cli() -> None:
     # subparser - command: clean
     subparser_cmd_clean = subparsers.add_parser(
         name="clean",
-        description="Remove all VMs and all VDIs on local storage from target pools",
-        help="Remove all VMs and all VDIs on local storage from target pools",
+        description="Remove all VMs, snapshorts and VDIs on local storage from target pools",
+        help="Remove all VMs, snapshorts and VDIs on local storage from target pools",
     )
     cmd_clean_excl_grp = subparser_cmd_clean.add_mutually_exclusive_group(required=True)
     cmd_clean_excl_grp.add_argument(

lib/tools/tasks/clean.py

@@ -59,6 +59,10 @@ def clean_pool(pool: Pool, dry_run: bool) -> None:
         'vm-list',
         {'is-control-domain': False, 'is-a-template': False},
         minimal=True,
+    )) + safe_split(master.xe(
+        'vm-list',
+        {'is-a-snapshot': True},
+        minimal=True,
     ))
     for vm_uuid in vm_uuids:
         vm = VM(vm_uuid, master)