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

Side by Side Diff: appengine/findit/waterfall/trigger_flake_swarming_task_pipeline.py

Issue 2130543004: Waterfall components of regression range finder. (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Created 4 years, 5 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 unified diff | Download patch
OLDNEW
(Empty)
1 from google.appengine.ext import ndb
2
3 from waterfall.trigger_base_swarming_task_pipeline import (
4 TriggerBaseSwarmingTaskPipeline as TBSTP)
5 from model.flake.flake_swarming_task import FlakeSwarmingTask
6
7 class TriggerFlakeSwarmingTaskPipeline(TBSTP):
8 """A pipeline to check if selected tests of a step are flaky.
9
10 This pipeline only supports test steps that run on Swarming and support the
11 gtest filter.
12 """
13
14 @ndb.transactional
stgao 2016/07/09 00:04:34 Why this is needed?
caiw 2016/07/14 00:59:45 Done.
15 def _GetSwarmingTask(self,master_name, builder_name, step_name, build_number):
16 # Get the appropriate kind of Swarming Task (Flake).
17 swarming_task = FlakeSwarmingTask.Get(
18 master_name, builder_name, step_name, build_number)
19 return swarming_task
20
21 @ndb.transactional
stgao 2016/07/09 00:04:34 Why this is needed too?
caiw 2016/07/14 00:59:45 Done.
22 def _CreateSwarmingTask(self, master_name, builder_name,
23 step_name, build_number):
24 # Create the appropriate kind of Swarming Task (Flake).
25 swarming_task = FlakeSwarmingTask.Create(
26 master_name, builder_name, step_name, build_number)
27 return swarming_task
28
29 def _GetIterationsToRerun(self):
30 # How many times we want to run the swarming rerun.
31 # Placeholder:
32 return 10
stgao 2016/07/09 00:04:34 How about adding a TODO here to move this into the
caiw 2016/07/14 00:59:45 Done.
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698