Exception in logdog_wrapper.py on Android gpu.fyi bots |
|||
Issue descriptionE.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
,
Aug 12 2016
Yuly works with Android stuff as well.
,
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
,
Aug 15 2016
Thanks for catching this Sunny. The bots are green again; is this considered fixed?
,
Aug 15 2016
Yes, this is fixed. E.g. this build: https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Release%20%28Nexus%205X%29/builds/1799 has the correct logdog links (e.g. https://luci-logdog.appspot.com/v/?s=chromium%2Fandroid%2Fswarming%2Flogcats%2F30a862f4c9942b11%2F%2B%2Funified_logcats for gl_tests) |
|||
►
Sign in to add a comment |
|||
Comment 1 by sunn...@chromium.org
, Aug 12 2016