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

Issue 637382 link

Starred by 3 users

Issue metadata

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



Sign in to add a comment

Exception in logdog_wrapper.py on Android gpu.fyi bots

Project Member Reported by sunn...@chromium.org, Aug 12 2016

Issue description

E.g. https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Release%20%28Nexus%205X%29/builds/1711

The angle_end2end_tests, angle_unittests, gl_tests and gl_unittests recipes are affected. There are no test failures but this error shows up:

Traceback (most recent call last):
  File "./../../build/android/test_wrapper/logdog_wrapper.py", line 66, in <module>
    sys.exit(main())
  File "./../../build/android/test_wrapper/logdog_wrapper.py", line 60, in main
    subprocess.call(logdog_cmd)
  File "/usr/lib/python2.7/subprocess.py", line 522, in call
    return Popen(*popenargs, **kwargs).wait()
  File "/usr/lib/python2.7/subprocess.py", line 710, in __init__
    errread, errwrite)
  File "/usr/lib/python2.7/subprocess.py", line 1327, in _execute_child
    raise child_exception
OSError: [Errno 2] No such file or directory

There was a recent change to how logcat is collected on for swarming tasks which seems to be the cause for this:
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/Y_4ctArAVfw

nicholaslin@ suggested that the gtest_tests (angle_end2end_tests, angle_unittests, gl_tests and gl_unittests) need to have these sections in the swarming dict:

  991           "cipd_packages": [
  992             {
  993               "cipd_package": "infra/tools/luci/logdog/butler/${platform}",
  994               "location": "bin",
  995               "revision": "git_revision:3ff24775a900b675866fbcacf2a8f98a18b2a16a
      "
  996             }
  997           ],

 1006           "output_links": [
 1007             {
 1008               "link": [
 1009                 "https://luci-logdog.appspot.com/v/?s",
 1010                 "=android%2Fswarming%2Flogcats%2F",
 1011                 "${TASK_ID}%2F%2B%2Funified_logcats"
 1012               ],
 1013               "name": "shard #${SHARD_INDEX} logcats"
 1014             }
 1015           ]

See https://codereview.chromium.org/2240923002/diff/20001/testing/buildbot/chromium.linux.json
 
Cc: geoffl...@chromium.org jmad...@chromium.org
Fix: https://codereview.chromium.org/2238333002

Also +geofflang, jmadill
Cc: ynovikov@chromium.org
Yuly works with Android stuff as well.
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 12 2016

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

commit 5296a496b951bb0d67a7e1d89106d442bd2174e0
Author: sunnyps <sunnyps@chromium.org>
Date: Fri Aug 12 21:48:30 2016

gpu: Fix gtest recipes to be compatible with unified swarming logcat.

TBR=kbr@chromium.org,nicholaslin@google.com
BUG= 637382 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel

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

[modify] https://crrev.com/5296a496b951bb0d67a7e1d89106d442bd2174e0/content/test/gpu/generate_buildbot_json.py
[modify] https://crrev.com/5296a496b951bb0d67a7e1d89106d442bd2174e0/testing/buildbot/chromium.gpu.fyi.json

Comment 4 by kbr@chromium.org, Aug 15 2016

Thanks for catching this Sunny. The bots are green again; is this considered fixed?

Sign in to add a comment