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

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

Issue 2652223002: Disable swarming for tests where it's disabled on builder level (FYI masters) (Closed)
Patch Set: review Created 3 years, 11 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.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 8aed8ca1fcc64f8104919a2a236f6da36c70a02b..8fda4769b78b647da8c8328d5ebe65bf53e3a522 100755
--- a/content/test/gpu/generate_buildbot_json.py
+++ b/content/test/gpu/generate_buildbot_json.py
@@ -1644,7 +1644,7 @@ def generate_gtest(tester_name, tester_config, test, test_config,
if not 'swarming' in result:
result['swarming'] = {}
result['swarming'].update({
- 'can_use_on_swarming_builders': True,
+ 'can_use_on_swarming_builders': tester_config['swarming'],
'dimension_sets': tester_config['swarming_dimensions']
})
if is_android(tester_config):
@@ -1753,7 +1753,7 @@ def generate_isolated_test(tester_name, tester_config, test, test_config,
swarming = {
# Always say this is true regardless of whether the tester
# supports swarming. It doesn't hurt.
Ken Russell (switch to Gerrit) 2017/01/25 22:28:09 Agree with Dirk's comment above. Please update thi
- 'can_use_on_swarming_builders': True,
+ 'can_use_on_swarming_builders': tester_config['swarming'],
'dimension_sets': tester_config['swarming_dimensions']
}
if 'swarming' in test_config:
« no previous file with comments | « no previous file | testing/buildbot/chromium.fyi.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698