commit beb40d83e9bd978eb99eb08b00f433e213353f35
author Mike Frysinger <vapier@chromium.org> Wed Aug 30 10:42:22 2023 -0400
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Aug 31 00:53:19 2023 +0000
tree 0aabc7085688ca39c6ae298b73db82488981207a
parent ce3d78f6a550664a7f4cc99c4b9a69a324a3d2d0

scripts: cros_sdk: improve delete logging

Make it clear what paths we're actually deleting (or not in the case
of out) in case the user didn't mean to do this thing.

BUG=b:289101910
TEST=CQ passes

Change-Id: If2a34a0481238709023e1270902e3e0afc4c5abe
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4826051
Commit-Queue: Brian Norris <briannorris@chromium.org>
Auto-Submit: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Brian Norris <briannorris@chromium.org>

scripts/cros_sdk.py

diff --git a/scripts/cros_sdk.py b/scripts/cros_sdk.py
index ff23d87..f6fe417 100644
--- a/scripts/cros_sdk.py
+++ b/scripts/cros_sdk.py
@@ -809,7 +809,12 @@
                     logging.warning(
                         "cros_sdk was invoked with force option, continuing."
                     )
-            logging.notice("Deleting chroot.")
+            logging.notice("Deleting chroot: %s", chroot.path)
+            logging.notice(
+                "%s output dir: %s",
+                "Deleting" if options.delete_out_dir else "Keeping",
+                chroot.out_path,
+            )
             cros_sdk_lib.CleanupChrootMount(
                 chroot, delete=True, delete_out=options.delete_out_dir
             )