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

Unified Diff: content/test/gpu/generate_buildbot_json.py

Issue 2238333002: gpu: Fix gtest recipes to be compatible with unified swarming logcat. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: only android Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | testing/buildbot/chromium.gpu.fyi.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/test/gpu/generate_buildbot_json.py
diff --git a/content/test/gpu/generate_buildbot_json.py b/content/test/gpu/generate_buildbot_json.py
index 39347db7d24252acb434dcc734b431339265cc41..c2b36f64261b5e9ba1554e8567c06019950782cc 100755
--- a/content/test/gpu/generate_buildbot_json.py
+++ b/content/test/gpu/generate_buildbot_json.py
@@ -1090,12 +1090,32 @@ def generate_gtest(tester_name, tester_config, test, test_config, is_fyi):
'can_use_on_swarming_builders': True,
'dimension_sets': [
tester_config['swarming_dimensions']
- ],
+ ]
})
if is_android(tester_config):
# Override the isolate target to get rid of any "_apk" suffix
# that would be added by the recipes.
result['override_isolate_target'] = test
+ # Integrate with the unified logcat system.
+ result['swarming'].update({
+ 'cipd_packages': [
+ {
+ 'cipd_package': 'infra/tools/luci/logdog/butler/${platform}',
+ 'location': 'bin',
+ 'revision': 'git_revision:3ff24775a900b675866fbcacf2a8f98a18b2a16a'
+ }
+ ],
+ 'output_links': [
+ {
+ 'link': [
+ 'https://luci-logdog.appspot.com/v/?s',
+ '=android%2Fswarming%2Flogcats%2F',
+ '${TASK_ID}%2F%2B%2Funified_logcats'
+ ],
+ 'name': 'shard #${SHARD_INDEX} logcats'
+ }
+ ]
+ })
if 'desktop_args' in result:
if not is_android(tester_config):
if not 'args' in result:
« no previous file with comments | « no previous file | testing/buildbot/chromium.gpu.fyi.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698