fix: show matched hosts when applying

adamkirchberger · Apr 18, 2022 · c9bf83b · c9bf83b
1 parent afacfd6
commit c9bf83b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/nectl/configs/cli.py b/nectl/configs/cli.py
@@ -171,9 +171,12 @@ def apply_cmd(
         print(f"Error: {e}")
         sys.exit(1)
 
+    print("Applying config to:")
+    print("\n".join([f"- {host.id}" for host in hosts]))
+
     if not assumeyes:
         click.confirm(
-            f"Are you sure you want to modify {len(hosts)} live hosts?", abort=True
+            f"\nAre you sure you want to modify {len(hosts)} live hosts?", abort=True
         )
 
     sys.exit(

diff --git a/nectl/datatree/hosts.py b/nectl/datatree/hosts.py
@@ -214,7 +214,10 @@ def is_match(host: Host) -> bool:
             # Add host
             hosts.append(host)
 
-    logger.info(f"hosts after filter: {len(hosts)}")
+    logger.info(
+        f"filter matched {len(hosts)} hosts: "
+        f"{','.join([host.id for host in hosts])}"
+    )
 
     if len(hosts) == 0:
         print("No hosts found.", file=sys.stderr)