commit 9c97f732c44daddc806ff9bbf018f259dd773348
author Sean Paul <seanpaul@chromium.org> Mon Jan 25 09:40:57 2016 -0500
committer chrome-bot <chrome-bot@chromium.org> Mon Jan 25 15:35:35 2016 -0800
tree 84be0d39494e4a9f550ab619dd1bf8540269d669
parent cd3babf90f4217ac273d81f437dfafbb60d64abf

dark_resume_utils: Don't ignore status on umount

Don't ignore the umount status in teardown, it could hide
issues on cleanup that affect the DUT moving forward.

BUG=None
TEST=Tested on samus with power_DarkResumeDisplay test

Change-Id: Icf5dcdcbaff40e8a0700051c062ae64c010d5e85
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/323481
Reviewed-by: Eric Caruso <ejcaruso@chromium.org>
Reviewed-by: Todd Broch <tbroch@chromium.org>

server/cros/dark_resume_utils.py

diff --git a/server/cros/dark_resume_utils.py b/server/cros/dark_resume_utils.py
index 83214f6..8897bbb 100644
--- a/server/cros/dark_resume_utils.py
+++ b/server/cros/dark_resume_utils.py
@@ -115,7 +115,7 @@
         logging.info('Tearing down dark resume preferences')
 
         logging.debug('Cleaning up temporary powerd bind mounts')
-        self._host.run('umount %s' % POWER_DIR, ignore_status=True)
+        self._host.run('umount %s' % POWER_DIR)
 
         logging.debug('Restarting powerd to revert to old settings')
         self._host.run('restart powerd')