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

Unified Diff: scripts/slave/recipes/chromium.py

Issue 1104533002: Add recipe for split AMP/local CQ. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Parent builder Android Builder (dbg) and expectation changes. Created 5 years, 6 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
Index: scripts/slave/recipes/chromium.py
diff --git a/scripts/slave/recipes/chromium.py b/scripts/slave/recipes/chromium.py
index dc196a800eb016d27109e257c3f6127653f1f459..05afb62895785b409796367bc3add164279bdde6 100644
--- a/scripts/slave/recipes/chromium.py
+++ b/scripts/slave/recipes/chromium.py
@@ -4,6 +4,7 @@
DEPS = [
'adb',
+ 'amp',
'bisect_tester',
'chromium',
'chromium_android',
@@ -15,9 +16,9 @@ DEPS = [
'properties',
'python',
'raw_io',
- 'test_utils',
'step',
'swarming',
+ 'test_utils',
]
@@ -456,3 +457,40 @@ def GenTests(api):
revision='1e74b372f951d4491f305ec64f6decfcda739e73') +
api.platform('win', 32)
)
+
+ yield (
+ api.test('amp_split_recipe_trigger_failure') +
+ api.properties(
+ mastername='chromium.fyi',
+ buildername='Android Tests (amp split)',
+ slavename='build1-a1',
+ buildnumber='77457',
+ parent_build_archive_url='gs://test-domain/test-archive.zip'
+ ) +
+ api.override_step_data('[trigger] base_unittests', retcode=1)
+ )
+
+ yield (
+ api.test('amp_split_recipe_trigger_local_failure') +
+ api.properties(
+ mastername='chromium.fyi',
+ buildername='Android Tests (amp split)',
+ slavename='build1-a1',
+ buildnumber='77457',
+ parent_build_archive_url='gs://test-domain/test-archive.zip'
+ ) +
+ api.override_step_data('[trigger] base_unittests', retcode=1) +
+ api.override_step_data('base_unittests', retcode=1)
+ )
+
+ yield (
+ api.test('amp_split_recipe_collect_failure') +
+ api.properties(
+ mastername='chromium.fyi',
+ buildername='Android Tests (amp split)',
+ slavename='build1-a1',
+ buildnumber='77457',
+ parent_build_archive_url='gs://test-domain/test-archive.zip'
+ ) +
+ api.override_step_data('[collect] base_unittests', retcode=1)
+ )

Powered by Google App Engine
This is Rietveld 408576698