Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(128)

Unified Diff: server/hosts/serial.py

Issue 3554003: Merge remote branch 'cros/upstream' into tempbranch3 (Closed) Base URL: http://git.chromium.org/git/autotest.git
Patch Set: Created 10 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « server/hosts/remote.py ('k') | server/server_job.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: server/hosts/serial.py
diff --git a/server/hosts/serial.py b/server/hosts/serial.py
index 9b4bdb24f3f861e67f645d4950db8d4091aa704a..d514dba33205d7671e8aa45c5c992f0f7ffdc535 100644
--- a/server/hosts/serial.py
+++ b/server/hosts/serial.py
@@ -114,7 +114,7 @@ class SerialHost(SiteHost):
def hardreset(self, timeout=DEFAULT_REBOOT_TIMEOUT, wait=True,
- conmux_command='hardreset', num_attempts=1,
+ conmux_command='hardreset', num_attempts=1, halt=False,
**wait_for_restart_kwargs):
"""
Reach out and slap the box in the power switch.
@@ -124,6 +124,7 @@ class SerialHost(SiteHost):
@params wait: Whether or not to wait for the machine to reboot
@params num_attempts: Number of times to attempt hard reset erroring
on the last attempt.
+ @params halt: Halts the machine before hardresetting.
@params **wait_for_restart_kwargs: keyword arguments passed to
wait_for_restart()
"""
@@ -138,6 +139,8 @@ class SerialHost(SiteHost):
old_boot_id = 'unknown boot_id prior to SerialHost.hardreset'
def reboot():
+ if halt:
+ self.halt()
if not self.run_conmux(conmux_command):
self.record("ABORT", None, "reboot.start",
"hard reset unavailable")
« no previous file with comments | « server/hosts/remote.py ('k') | server/server_job.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698