New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 707387 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

android trybots do not save device logcat anymore

Project Member Reported by boliu@chromium.org, Mar 31 2017

Issue description

linux_android_rel_ng and android_n5x_swarming_rel
 
Owner: hzl@chromium.org
Status: Assigned (was: Untriaged)
I think there should now be a link at the bottom of a test's output that will take you to the logcats. It should look something like "https://luci-logdog.appspot.com/v/?s=chromium%2Fandroid%2Fswarming%2Flogcats%..."

Comment 2 by boliu@chromium.org, Mar 31 2017

err, still not sure where that link is.. where is that link on this run?
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_android_rel_ng/builds/262568
Say you wanted the logcats for components_unittests, you'd click the stdout link underneath that test (step #56), scroll to the bottom, and there's this line of text:

C  733.050s Main  Logcat: https://luci-logdog.appspot.com/v/?s=chromium%2Fandroid%2Fswarming%2Flogcats%2F353f0a7c08dd1311%2F%2B%2Funified_logcats

Comment 4 by boliu@chromium.org, Mar 31 2017

cool, so looks like it's very close to actually surfacing that link directly in the buildbot page. but this intermediate state is so much worse. can we make sure these changes are communicated with developers?
Labels: -Pri-3 Pri-2
Project Member

Comment 6 by bugdroid1@chromium.org, Apr 4 2017

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

commit 9ae14459caaa7f3b049b3d44be2c209890773213
Author: hzl <hzl@google.com>
Date: Tue Apr 04 23:38:02 2017

Save the logcat file in isolated directory.

Previously we store the logcat into a temporary directory, which will be
deleted shortly. That means logcat will not be viewable in the isolated
directory.

In this cl, we are storing the logcat in the isolated directory, for
later easier view. This is a temporary fix before result details is
enabled.

BUG= 707387 

Review-Url: https://codereview.chromium.org/2797773003
Cr-Commit-Position: refs/heads/master@{#461898}

[modify] https://crrev.com/9ae14459caaa7f3b049b3d44be2c209890773213/build/android/test_wrapper/logdog_wrapper.py
[modify] https://crrev.com/9ae14459caaa7f3b049b3d44be2c209890773213/tools/mb/mb.py

Comment 7 by hzl@chromium.org, Apr 6 2017

Status: Fixed (was: Assigned)
Thank bo so much for taking the time to report this issue. I believe the bug is fixed now and logcat is visible in isolated out now.

Sign in to add a comment