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

Unified Diff: appengine/findit/waterfall/waterfall_config.py

Issue 1898493002: [Findit] Adding support for disabling test try jobs for given master/builder (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Addressing comments Created 4 years, 8 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: appengine/findit/waterfall/waterfall_config.py
diff --git a/appengine/findit/waterfall/waterfall_config.py b/appengine/findit/waterfall/waterfall_config.py
index 6fb5ee1fe7da166c8ba54242b31873326e02809b..9a7a6a87a5ed438a252bc5d1616691256dedf3d5 100644
--- a/appengine/findit/waterfall/waterfall_config.py
+++ b/appengine/findit/waterfall/waterfall_config.py
@@ -137,6 +137,24 @@ def EnableStrictRegexForCompileLinkFailures(wf_mastername, wf_buildername):
return trybot_config.get('strict_regex', False)
+def ShouldSkipTestTryJobs(wf_mastername, wf_buildername):
+ """Returns True if test try jobs should be triggered.
+
+ By default, test try jobs should be supported unless the master/builder
+ specifies to bail out.
+
+ Args:
+ wf_mastername: The mastername of a waterfall builder.
+ wf_buildername: The buildername of a waterfall builder.
+
+ Returns:
+ True if test try jobs are to be skipped, False otherwise.
+ """
+ trybot_config = FinditConfig.Get().builders_to_trybots.get(
+ wf_mastername, {}).get(wf_buildername, {})
+ return trybot_config.get('not_run_tests', False)
+
+
def GetTryJobSettings():
return FinditConfig().Get().try_job_settings
« appengine/findit/waterfall/try_job_util.py ('K') | « appengine/findit/waterfall/try_job_util.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698