Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(188)

Issue 23601007: Revert 220166 "Archive webkit_tests results on android bots if r..." (Closed)

Created:
7 years, 3 months ago by tkent
Modified:
7 years, 3 months ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 220166 "Archive webkit_tests results on android bots if r..." Android bot throws an exception since this change. < cat /b/build/slave/WebKit_Android__Nexus4_/build/src/out/Release/test_logs/unit_test.log An instance of host heart beart running... will kill Traceback (most recent call last): File "build/android/buildbot/bb_device_steps.py", line 469, in <module> sys.exit(main(sys.argv)) File "build/android/buildbot/bb_device_steps.py", line 465, in main MainTestWrapper(options) File "build/android/buildbot/bb_device_steps.py", line 402, in MainTestWrapper bb_utils.RunSteps(options.test_filter, GetTestStepCmds(), options) File "/b/build/slave/WebKit_Android__Nexus4_/build/src/build/android/buildbot/bb_utils.py", line 97, in RunSteps cmd(options) File "build/android/buildbot/bb_device_steps.py", line 257, in RunWebkitLayoutTests '--builder-name', options.build_properties.get('buildername', '')]) File "/b/build/slave/WebKit_Android__Nexus4_/build/src/build/android/buildbot/bb_utils.py", line 56, in RunCmd code = SpawnCmd(command, stdout, cwd).wait() File "/b/build/slave/WebKit_Android__Nexus4_/build/src/build/android/buildbot/bb_utils.py", line 49, in SpawnCmd return subprocess.Popen(command, cwd=cwd, stdout=stdout) File "/usr/lib/python2.7/subprocess.py", line 679, in __init__ errread, errwrite) File "/usr/lib/python2.7/subprocess.py", line 1249, in _execute_child raise child_exception OSError: [Errno 2] No such file or directory > Archive webkit_tests results on android bots if requested. > > R=ilevy@chromium.org, bulach@chromium.org, peter@chromium.org > BUG=276076 > > Review URL: https://chromiumcodereview.appspot.com/22825016 TBR=dpranke@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=220210

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -27 lines) Patch
M trunk/src/build/android/buildbot/bb_device_steps.py View 7 chunks +12 lines, -24 lines 0 comments Download
M trunk/src/build/android/buildbot/bb_utils.py View 1 chunk +0 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
tkent
7 years, 3 months ago (2013-08-29 05:15:24 UTC) #1
tkent
Committed patchset #1 manually as r220210.
7 years, 3 months ago (2013-08-29 05:15:33 UTC) #2
Dirk Pranke
7 years, 3 months ago (2013-08-29 06:24:05 UTC) #3
Message was sent while issue was closed.
On 2013/08/29 05:15:33, tkent wrote:
> Committed patchset #1 manually as r220210.

Thanks for reverting! I'll take a look at what was wrong.

lgtm.

Powered by Google App Engine
This is Rietveld 408576698