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

Issue 652337 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Impose strict timeouts on all forwarder invocations

Project Member Reported by jbudorick@chromium.org, Oct 3 2016

Issue description

Test runs should not hang indefinitely due to bugs in the forwarder.

Additional context: https://bugs.chromium.org/p/chromium/issues/detail?id=652250
 
Cc: -jbudorick@chromium.org
Owner: jbudorick@chromium.org
Status: Started (was: Available)
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 5 2016

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

commit aec33252c12a0dfc6e768fce18739be89b38412e
Author: catapult-deps-roller <catapult-deps-roller@chromium.org>
Date: Wed Oct 05 19:13:55 2016

Roll src/third_party/catapult/ f68d5268f..d489dd8da (2 commits).

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/f68d5268f877..d489dd8dabbf

$ git log f68d5268f..d489dd8da --date=short --no-merges --format='%ad %ae %s'
2016-10-05 hjd Add a method that backgrounds the browser
2016-10-05 jbudorick [devil] Run all forwarder commands with a timeout.

BUG= 586148 , 652337 

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=catapult-sheriff@chromium.org

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

[modify] https://crrev.com/aec33252c12a0dfc6e768fce18739be89b38412e/DEPS

Status: Fixed (was: Started)
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 27 2016

Labels: merge-merged-2840
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/aec33252c12a0dfc6e768fce18739be89b38412e

commit aec33252c12a0dfc6e768fce18739be89b38412e
Author: catapult-deps-roller <catapult-deps-roller@chromium.org>
Date: Wed Oct 05 19:13:55 2016

Roll src/third_party/catapult/ f68d5268f..d489dd8da (2 commits).

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/f68d5268f877..d489dd8dabbf

$ git log f68d5268f..d489dd8da --date=short --no-merges --format='%ad %ae %s'
2016-10-05 hjd Add a method that backgrounds the browser
2016-10-05 jbudorick [devil] Run all forwarder commands with a timeout.

BUG= 586148 , 652337 

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=catapult-sheriff@chromium.org

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

[modify] https://crrev.com/aec33252c12a0dfc6e768fce18739be89b38412e/DEPS

Comment 5 by dimu@google.com, Nov 4 2016

Labels: -merge-merged-2840
[Automated comment] removing mislabelled merge-merged-2840

Sign in to add a comment