commit b2919e8cbc5a1a19b0fc5448e55a6bff478c7ed9
author Alex Klein <saklein@chromium.org> Wed Feb 23 10:55:36 2022 -0700
committer Commit Bot <commit-bot@chromium.org> Sat Feb 26 00:28:28 2022 +0000
tree 141dfdadc9398428a19711282755c1a2878db410
parent 6fb84cd043663b74dd44f559a5f28982b4b97a67

controller/sdk: Switch sdk clean back to using safe.

The --flash option (included in safe) no longer enters the chroot to
clean its directory. Switch the SdkService/Clean back to sysroots + safe.

BUG=b:218332920
TEST=./run_tests

Change-Id: Ib0033932d07b3aa4fce31f4b9db7ebe8d9da5b45
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/3485224
Tested-by: Alex Klein <saklein@chromium.org>
Reviewed-by: Lizzy Presland <zland@google.com>
Commit-Queue: Alex Klein <saklein@chromium.org>

api/controller/sdk.py

diff --git a/api/controller/sdk.py b/api/controller/sdk.py
index b6a8fad..160ba88 100644
--- a/api/controller/sdk.py
+++ b/api/controller/sdk.py
@@ -121,14 +121,7 @@
 def Clean(input_proto, _output_proto, _config):
   """Clean unneeded files from a chroot."""
   chroot = controller_util.ParseChroot(input_proto.chroot)
-  sdk.Clean(
-      chroot,
-      images=True,
-      sysroots=True,
-      tmp=True,
-      cache=True,
-      logs=True,
-      workdirs=True)
+  sdk.Clean(chroot, safe=True, sysroots=True)
 
 
 @faux.all_empty