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

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

Issue 2508603002: [Findit] Refactoring trigger swarming task pipelines (Closed)
Patch Set: Created 4 years, 1 month 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/trigger_flake_swarming_task_pipeline.py
diff --git a/appengine/findit/waterfall/trigger_flake_swarming_task_pipeline.py b/appengine/findit/waterfall/trigger_flake_swarming_task_pipeline.py
index e6abcba7979df74f570c0a2c410d3576de6085db..9b4f95ce6b8e895e7455ac652331e4ade2053650 100644
--- a/appengine/findit/waterfall/trigger_flake_swarming_task_pipeline.py
+++ b/appengine/findit/waterfall/trigger_flake_swarming_task_pipeline.py
@@ -15,6 +15,10 @@ class TriggerFlakeSwarmingTaskPipeline(TriggerBaseSwarmingTaskPipeline):
gtest filter.
"""
+ def _GetArgs(self, master_name, builder_name, build_number, step_name, tests):
+ test_name = tests[0] # Only one test per pipeline.
+ return (master_name, builder_name, build_number, step_name, test_name)
+
# pylint: disable=arguments-differ
def _GetSwarmingTask(self, master_name, builder_name, build_number,
step_name, test_name):
@@ -33,7 +37,3 @@ class TriggerFlakeSwarmingTaskPipeline(TriggerBaseSwarmingTaskPipeline):
def _GetIterationsToRerun(self):
return waterfall_config.GetCheckFlakeSettings().get('iterations_to_rerun')
-
- def _GetArgs(self, master_name, builder_name, build_number, step_name, tests):
- test_name = tests[0] # Only one test per pipeline.
- return (master_name, builder_name, build_number, step_name, test_name)

Powered by Google App Engine
This is Rietveld 408576698