Commit ed7e28aa authored by Sergey Ulanov's avatar Sergey Ulanov Committed by Commit Bot

[Fuchsia] Don't close qemu stderr

After https://crrev.com/580342 fuchsia runner scripts redirects system
log to serial port and captures if from qemu stdout. It also closes
the stdout stream after the system is booted. In some cases this
causes qemu die later. With this change the stdout pipe is redirected
to /dev/null instead of being closed.

Change-Id: I37bcc744d4ca0cb72ac1dffd3d436c3675c951eb
Reviewed-on: https://chromium-review.googlesource.com/1162864
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: default avatarWez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580728}
parent 56a3351b
...@@ -167,9 +167,10 @@ class Target(object): ...@@ -167,9 +167,10 @@ class Target(object):
return True return True
time.sleep(_ATTACH_RETRY_INTERVAL) time.sleep(_ATTACH_RETRY_INTERVAL)
finally: finally:
# Redirect logs to /dev/null. run_package.py will use SSH+dlog to get
# logs from the machine to console.
if self._system_logs_reader: if self._system_logs_reader:
self._system_logs_reader.Close() self._system_logs_reader.RedirectTo(open('/dev/null', 'w'));
self._system_logs_reader = None
logging.error('Timeout limit reached.') logging.error('Timeout limit reached.')
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment