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

Unified Diff: scripts/slave/recipe_modules/bot_update/example.py

Issue 1382203002: bot_update (+recipe_module): treat patch download as infra failure. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: NBits\ Created 5 years, 2 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/recipe_modules/bot_update/example.py
diff --git a/scripts/slave/recipe_modules/bot_update/example.py b/scripts/slave/recipe_modules/bot_update/example.py
index 7997900b2adddce0466cf25c4f4b4488e7404df1..f191f463cffaa3070c069911317334964dfb2be0 100644
--- a/scripts/slave/recipe_modules/bot_update/example.py
+++ b/scripts/slave/recipe_modules/bot_update/example.py
@@ -83,8 +83,17 @@ def GenTests(api):
issue=12345,
patchset=654321,
patch_url='http://src.chromium.org/foo/bar',
- fail_patch=True,
+ fail_patch='apply',
) + api.step_data('bot_update', retcode=88)
+ yield api.test('tryjob_fail_patch_download') + api.properties(
+ mastername='tryserver.chromium.linux',
+ buildername='linux_rel',
+ slavename='totallyaslave-c4',
+ issue=12345,
+ patchset=654321,
+ patch_url='http://src.chromium.org/foo/bar',
+ fail_patch='download'
+ ) + api.step_data('bot_update', retcode=87)
yield api.test('forced') + api.properties(
mastername='experimental',
buildername='Experimental Builder',

Powered by Google App Engine
This is Rietveld 408576698