Increase default level of feedback/confirmation in destructive commands

#1264 (#1283) Co-authored-by: stas <statis@microsoft.com>
microsoft · Sep 27, 2021 · d3bcbea · d3bcbea
1 parent 09bdad4
commit d3bcbea
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):