commit 19eccc58f34e8f5a529fbc98358d2f078f838cf7
author Otabek Kasimov <otabek@google.com> Tue Oct 20 12:22:53 2020 -0700
committer Commit Bot <commit-bot@chromium.org> Wed Oct 21 00:29:25 2020 +0000
tree 0e28c21d3f9f92782658f0b47bc58b4045b8d9a6
parent 41ea43e87107dfe51405ce2bc389758c3c8167e1

storage: Stop mark NVMe storage based on SMART stats

BUG=chromium:1140507
TEST=run local audit task

./site_utils/admin_audit/main.py --hostname chromeos1-row4-rack4-host3 --results-dir /tr --host-info-file /tr/host_info_store/chromeos1-row4-rack4-host3.store  verify-dut-storage

Change-Id: Ia182a46f52fe83f22fedb25b4a2eb0438fd4181a
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/2487862
Commit-Queue: Otabek Kasimov <otabek@google.com>
Commit-Queue: Garry Wang <xianuowang@chromium.org>
Tested-by: Otabek Kasimov <otabek@google.com>
Auto-Submit: Otabek Kasimov <otabek@google.com>
Reviewed-by: Garry Wang <xianuowang@chromium.org>

server/cros/storage/storage_validate.py

diff --git a/server/cros/storage/storage_validate.py b/server/cros/storage/storage_validate.py
index 06386b7..95b97b9 100644
--- a/server/cros/storage/storage_validate.py
+++ b/server/cros/storage/storage_validate.py
@@ -256,9 +256,9 @@
             raise StorageError('Storage state cannot be detected')
         if used_value < 91:
             return STORAGE_STATE_NORMAL
-        if used_value < 99:
-            return STORAGE_STATE_WARNING
-        return STORAGE_STATE_CRITICAL
+        # Stop mark device as bad when they reached 100% usage
+        # TODO(otabek) crbug.com/1140507 re-evaluate the max usage
+        return STORAGE_STATE_WARNING
 
     def _get_storage_path(self):
         """Find and return the path to the device storage.