New issue
Advanced search Search tips

Issue 620723 link

Starred by 1 user
Status: Fixed
Owner:
jbudorick@chromium.org
Closed: Jun 2016
Cc:
jbudorick@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Make the test runner handle SIGTERM

Project Member Reported by jbudorick@chromium.org, Jun 16 2016 
Swarming sends SIGTERM to child processes on timeout. //build/android/test_runner.py doesn't currently do anything with this. It should:
 - stop running tests
 - attempt orderly shut down w/ expected artifacts -- json results file, logcats, etc -- generated
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 16 2016 

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

commit 5507469bde2ccd9b46ffe5df1ec19369942346dc
Author: catapult-deps-roller <catapult-deps-roller@chromium.org>
Date: Thu Jun 16 23:20:19 2016

Roll src/third_party/catapult/ 62a4cd70a..b2bcbcedc (1 commit).

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/62a4cd70a91c..b2bcbcedccff

$ git log 62a4cd70a..b2bcbcedc --date=short --no-merges --format='%ad %ae %s'

BUG= 620723 

TBR=catapult-sheriff@chromium.org

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

[modify] https://crrev.com/5507469bde2ccd9b46ffe5df1ec19369942346dc/DEPS
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 17 2016 

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

commit 5dc7bbe9e52fd447f0c54d52d671c7d00c962ea5
Author: jbudorick <jbudorick@chromium.org>
Date: Fri Jun 17 15:17:02 2016

[Android] Stop gtests & instrumentation tests on receiving SIGTERM.

BUG= 620723 

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

[modify] https://crrev.com/5dc7bbe9e52fd447f0c54d52d671c7d00c962ea5/build/android/pylib/local/device/local_device_test_run.py
[modify] https://crrev.com/5dc7bbe9e52fd447f0c54d52d671c7d00c962ea5/build/android/test_runner.pydeps

Comment 3 by jbudorick@chromium.org, Jun 17 2016

Status: Fixed (was: Assigned)
►

Sign in to add a comment