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

Issue 623743 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Feature



Sign in to add a comment

Enable crash reporting for ADB host native crashes

Project Member Reported by bryanlew@chromium.org, Jun 27 2016

Issue description

Enable crash reporting for ADB hosts for native crashes (not Java crashes).

Split off from:
https://bugs.chromium.org/p/chromium/issues/detail?id=605623
 
Status: Started (was: Available)
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 10 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/11f8aedd8f1577ca7a94b7cf687064138fe35896

commit 11f8aedd8f1577ca7a94b7cf687064138fe35896
Author: Bryan Lewandowski <bryanlew@google.com>
Date: Wed Jun 29 22:44:40 2016

Add crash log collection for Brillo DUTs.

Added hooks for collecting native crash logs from ADBHost hosts.
Currently this is only available on Brillo.  A test was added to
check that Brillo crash logs are created when a process is killed.

BUG= chromium:623743 
TEST=test_droid against local device
TEST=test_droid against remote device
TEST=New test run with test_droid on both local and remote devices

Change-Id: I90d59a64e8485310051679c1897b475628d617c3
Reviewed-on: https://chromium-review.googlesource.com/357721
Commit-Ready: Bryan Lewandowski <bryanlew@google.com>
Tested-by: Bryan Lewandowski <bryanlew@google.com>
Reviewed-by: Kevin Cheng <kevcheng@chromium.org>

[add] https://crrev.com/11f8aedd8f1577ca7a94b7cf687064138fe35896/server/site_tests/brillo_CrashLogging/control
[add] https://crrev.com/11f8aedd8f1577ca7a94b7cf687064138fe35896/server/site_tests/brillo_CrashLogging/brillo_CrashLogging.py
[modify] https://crrev.com/11f8aedd8f1577ca7a94b7cf687064138fe35896/server/hosts/adb_host.py

Status: Verified (was: Started)
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 19 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/6708ea672c2e0b9a86b27e8e6c964ee99f9faa39

commit 6708ea672c2e0b9a86b27e8e6c964ee99f9faa39
Author: Bryan Lewandowski <bryanlew@google.com>
Date: Mon Jul 18 22:09:06 2016

Disable/revert log collection while we look into an issue in Brillo.

BUG= chromium:623743 
TEST=Local test_droid on Brillo and Android log/dummy tests.

Change-Id: Ib1189d0dfcba08576fc2ae39108d463bea4fd222
Reviewed-on: https://chromium-review.googlesource.com/361375
Commit-Ready: Bryan Lewandowski <bryanlew@google.com>
Tested-by: Bryan Lewandowski <bryanlew@google.com>
Reviewed-by: Ralph Nathan <ralphnathan@chromium.org>

[modify] https://crrev.com/6708ea672c2e0b9a86b27e8e6c964ee99f9faa39/server/hosts/adb_host.py

Project Member

Comment 5 by bugdroid1@chromium.org, Jul 26 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/8c96bb2b1eccf3bbfff88efb7cbed5bcf78b09c9

commit 8c96bb2b1eccf3bbfff88efb7cbed5bcf78b09c9
Author: Bryan Lewandowski <bryanlew@google.com>
Date: Tue Jul 19 19:56:03 2016

Reenable logging, warn on logging failure.

Reenables native crash logging, and considers the inability to
enable crash logging as a warning rather than an error.  This
is important because the provision operation calls job_start(),
and depending on the preexisting image on the DUT, the attempt
to enable logging will fail.  Note that job_start() is called
again to start the test case, so logging will be enabled once
the device is provisioned with the image under test.

BUG= chromium:623743 
BUG= chromium:627134 
TEST=Local test_droid on Brillo: android_DummyTest brillo_CrashLogging
TEST=Local test_droid on Android: android_DummyTest android_CrashLogging
TEST=Lab test_droid on Brillo: android_DummyTest brillo_CrashLogging
TEST=Guado MobLab on Brillo: android_DummyTest brillo_CrashLogging

Change-Id: I1da04f7f2c7c5133ea7befaf152afb4e2166f406
Reviewed-on: https://chromium-review.googlesource.com/362611
Commit-Ready: Bryan Lewandowski <bryanlew@google.com>
Tested-by: Bryan Lewandowski <bryanlew@google.com>
Reviewed-by: Simran Basi <sbasi@chromium.org>

[modify] https://crrev.com/8c96bb2b1eccf3bbfff88efb7cbed5bcf78b09c9/server/hosts/adb_host.py

Sign in to add a comment