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

Unified Diff: PRESUBMIT.py

Issue 772963003: Add android + chromeos + debug trybots to default set (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 6 years 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: PRESUBMIT.py
diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index ae483e83e80df47ffa36a6c5cf947ae0cdb53cfa..c07514659c752d9daf9efaf74b38a430dc529cbc 100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -1551,14 +1551,12 @@ def GetDefaultTryConfigs(bots=None):
"""
builders_and_tests = {
- # TODO(maruel): Figure out a way to run 'sizes' where people can
- # effectively update the perf expectation correctly. This requires a
- # clobber=True build running 'sizes'. 'sizes' is not accurate with
- # incremental build. Reference:
- # http://chromium.org/developers/tree-sheriffs/perf-sheriffs.
- # TODO(maruel): An option would be to run 'sizes' but not count a failure
- # of this step as a try job failure.
'Mojo Linux Try': ['defaulttests'],
+ 'Mojo Linux (dbg) Try': ['defaulttests'],
+ 'Mojo Android Builder Try': ['defaulttests'],
+ 'Mojo Android Builder (dbg) Try': ['defaulttests'],
+ 'Mojo ChromeOS Builder Try': ['defaulttests'],
+ 'Mojo ChromeOS Builder (dbg) Try': ['defaulttests'],
}
if bots:
@@ -1595,6 +1593,11 @@ def GetPreferredTryMasters(project, change):
builders = [
'Mojo Linux Try',
+ 'Mojo Linux (dbg) Try',
+ 'Mojo Android Builder Try',
+ 'Mojo Android Builder (dbg) Try',
+ 'Mojo ChromeOS Builder Try',
+ 'Mojo ChromeOS Builder (dbg) Try',
]
return GetDefaultTryConfigs(builders)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698