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

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

Issue 2243673002: [Findit] Added algorithm to analysis (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: gclient sync Created 4 years, 4 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
« no previous file with comments | « appengine/findit/waterfall/flake/test/recursive_flake_pipeline_test.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/findit/waterfall/process_base_swarming_task_result_pipeline.py
diff --git a/appengine/findit/waterfall/process_base_swarming_task_result_pipeline.py b/appengine/findit/waterfall/process_base_swarming_task_result_pipeline.py
index 9f07c8b0c623fecbb5e2f20f212d03692a6a9998..8b566bfabf53ad8b421defb07ac564a6682a1fc2 100644
--- a/appengine/findit/waterfall/process_base_swarming_task_result_pipeline.py
+++ b/appengine/findit/waterfall/process_base_swarming_task_result_pipeline.py
@@ -10,7 +10,6 @@ import time
from common.http_client_appengine import HttpClientAppengine as HttpClient
from common.pipeline_wrapper import BasePipeline
from model import analysis_status
-from model.wf_swarming_task import WfSwarmingTask
from waterfall import swarming_util
from waterfall import waterfall_config
@@ -28,7 +27,7 @@ class ProcessBaseSwarmingTaskResultPipeline(BasePipeline):
# Checks result status for each test run and saves the numbers accordingly.
# Should be overridden by subclass.
raise NotImplementedError(
- '_CheckTestsRunStatuses should be implemented in the child class')
+ '_CheckTestsRunStatuses should be implemented in the child class')
def _ConvertDateTime(self, time_string):
"""Convert UTC time string to datetime.datetime."""
@@ -47,7 +46,7 @@ class ProcessBaseSwarmingTaskResultPipeline(BasePipeline):
# Get the appropriate kind of Swarming Task (Wf or Flake).
# Should be overwritten by subclass.
raise NotImplementedError(
- '_GetSwarmingTask should be implemented in the child class')
+ '_GetSwarmingTask should be implemented in the child class')
def _GetArgs(self):
# Return list of arguments to call _CheckTestsRunStatuses with - output_json
@@ -57,7 +56,7 @@ class ProcessBaseSwarmingTaskResultPipeline(BasePipeline):
# Arguments number differs from overridden method - pylint: disable=W0221
def run(self, master_name, builder_name, build_number,
- step_name, task_id, *args): #pragma: no cover.
+ step_name, task_id, *args): # pragma: no cover.
"""
Args:
master_name (str): The master name.
« no previous file with comments | « appengine/findit/waterfall/flake/test/recursive_flake_pipeline_test.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698