Increase default level of feedback/confirmation in destructive commands

microsoft#1264
stishkin · Sep 24, 2021 · 1480322 · 1480322
1 parent d13ce06
commit 1480322
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/src/cli/onefuzz/api.py b/src/cli/onefuzz/api.py
@@ -1059,13 +1059,16 @@ def delete(
         for container_name in to_keep:
             self.logger.info("not removing: %s", container_name)
 
-        for container_name in to_delete:
-            if dryrun:
-                self.logger.info("container would be deleted: %s", container_name)
-            elif self.onefuzz.containers.delete(container_name).result:
-                self.logger.info("removed container: %s", container_name)
-            else:
-                self.logger.info("container already removed: %s", container_name)
+        if len(to_delete) > 0:
+            for container_name in to_delete:
+                if dryrun:
+                    self.logger.info("container would be deleted: %s", container_name)
+                elif self.onefuzz.containers.delete(container_name).result:
+                    self.logger.info("removed container: %s", container_name)
+                else:
+                    self.logger.info("container already removed: %s", container_name)
+        else:
+            self.logger.info("nothing to delete")
 
 
 class JobTasks(Endpoint):