commit f68f78966a28d7f6f1145beba8763a48af25f38a
author Meng-Huan Yu <menghuan@chromium.org> Fri Mar 06 19:59:27 2020 +0800
committer Commit Bot <commit-bot@chromium.org> Mon Mar 09 11:13:17 2020 +0000
tree 8cd1cdeea265b434c5839e691d8e62d50a311108
parent bf2d4ab9f3b41a7bdeee1e11c69c6ac9a155dc14

toolkit: Remove factory logs under /var/log when uninstalling

BUG=b:150189948
TEST=verified the files are removed on DUT

Change-Id: Ia5b3f533a00aefe05eb954287c65d15752a38d9e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/factory/+/2089167
Reviewed-by: Yilin Yang (kerker) <kerker@chromium.org>
Commit-Queue: Meng-Huan Yu <menghuan@chromium.org>
Tested-by: Meng-Huan Yu <menghuan@chromium.org>
Auto-Submit: Meng-Huan Yu <menghuan@chromium.org>

py/toolkit/uninstaller.py

diff --git a/py/toolkit/uninstaller.py b/py/toolkit/uninstaller.py
index 60e148b..901b5cf 100755
--- a/py/toolkit/uninstaller.py
+++ b/py/toolkit/uninstaller.py
@@ -28,7 +28,11 @@
 Uninstall the factory toolkit on a CrOS device.
 """
 
-FACTORY_DIRS = ['/var/factory', '/run/factory', paths.FACTORY_DIR]
+FACTORY_PATHS = [
+    '/var/factory', '/run/factory', paths.FACTORY_DIR,
+    '/var/log/factory-init.log', '/var/log/factory-session.log',
+    '/var/log/factory.log'
+]
 
 
 def AssertEnvironment():
@@ -49,7 +53,7 @@
       '\n'
       '*** You are about to uninstall the factory toolkit at:\n')
 
-  for p in FACTORY_DIRS:
+  for p in FACTORY_PATHS:
     ret += '***   %s\n' % p
 
   ret += '***'
@@ -84,7 +88,7 @@
                                    '/usr/local/bin', install_symlinks.MODE_MINI)
 
   # Delete all factory related files.
-  for p in FACTORY_DIRS:
+  for p in FACTORY_PATHS:
     process_utils.Spawn(['rm', '-rf', p], check_call=True, log=True)