New issue
Advanced search Search tips

Issue 749242 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Feb 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: Fuchsia
Pri: 3
Type: Bug



Sign in to add a comment

On fuchsia device runs, output is not returned to host

Project Member Reported by scottmg@chromium.org, Jul 26 2017

Issue description

https://chromium-review.googlesource.com/c/585411/ adds device runs via bootserver.

There's no way in the current setup to get the output back to the host (for symbolization, bot runs, etc.)

One way to accomplish this might be set up sshd, and then run the commands over ssh rather than via the autorun script.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/05b3b6349f8febd0d4bbda554962514c1913876e

commit 05b3b6349f8febd0d4bbda554962514c1913876e
Author: Scott Graham <scottmg@chromium.org>
Date: Thu Jul 27 00:26:50 2017

fuchsia: Support running on hardware

Adds a '--device' flag to Fuchsia run scripts. This causes the script
to run the Fuchsia SDK's bootserver instead of launching QEMU. If a
configured device is then connected to the bootserver the kernel and
boot image are sent to the device, and the tests run by autorun, the
same as in the QEMU case. (Press Alt-F1 to switch to 'debuglog' to see
the output on-device.)

Bug:  707029 , 706592,  749242 
Change-Id: I0a84885232463ee4cf316c6399edd6155ce97536
Reviewed-on: https://chromium-review.googlesource.com/585411
Commit-Queue: Scott Graham <scottmg@chromium.org>
Reviewed-by: Kevin Marshall <kmarshall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#489812}
[modify] https://crrev.com/05b3b6349f8febd0d4bbda554962514c1913876e/build/fuchsia/test_runner.py

Owner: kmarshall@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 23 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/552a014401109b3ca3ee03f8b0c99d1de808ddb5

commit 552a014401109b3ca3ee03f8b0c99d1de808ddb5
Author: Kevin Marshall <kmarshall@chromium.org>
Date: Mon Oct 23 21:59:38 2017

Fuchsia: add cross-device log handling to runner scripts.

This will streams executables' output from the remote device to the
terminal. It will symbolize any stack traces it encounters, just like
it does in QEMU currently.

Bug:  749242 
Change-Id: I3d1ffea0d3c642c46d32e3466c05e97938ff5fe7
Reviewed-on: https://chromium-review.googlesource.com/734128
Reviewed-by: Scott Graham <scottmg@chromium.org>
Commit-Queue: Kevin Marshall <kmarshall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#510932}
[modify] https://crrev.com/552a014401109b3ca3ee03f8b0c99d1de808ddb5/build/fuchsia/exe_runner.py
[modify] https://crrev.com/552a014401109b3ca3ee03f8b0c99d1de808ddb5/build/fuchsia/runner_common.py
[modify] https://crrev.com/552a014401109b3ca3ee03f8b0c99d1de808ddb5/build/fuchsia/test_runner.py

Verified on ToT
Project Member

Comment 5 by bugdroid1@chromium.org, Oct 24 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/15bcbbf9c315758685a818e7071aa31265dfb9c8

commit 15bcbbf9c315758685a818e7071aa31265dfb9c8
Author: Kevin Marshall <kmarshall@chromium.org>
Date: Tue Oct 24 01:16:51 2017

Fuchsia: use "dm reboot" after executing binaries on hardware.

The current code runs "dm poweroff" which now switches off the device.
It adds unnecessary friction for repeated runs, so this CL changes it
to "dm reboot" instead.


Bug:  749242 
Change-Id: Ic65e5b222560ede892e2193977c303d097de7a2b
Reviewed-on: https://chromium-review.googlesource.com/734333
Commit-Queue: Kevin Marshall <kmarshall@chromium.org>
Reviewed-by: Scott Graham <scottmg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#510977}
[modify] https://crrev.com/15bcbbf9c315758685a818e7071aa31265dfb9c8/build/fuchsia/exe_runner.py
[modify] https://crrev.com/15bcbbf9c315758685a818e7071aa31265dfb9c8/build/fuchsia/runner_common.py
[modify] https://crrev.com/15bcbbf9c315758685a818e7071aa31265dfb9c8/build/fuchsia/test_runner.py

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)

Sign in to add a comment