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

Unified Diff: appengine/findit/waterfall/flake/step_mapper.py

Issue 2396283002: [Findit] Hook up analysis for CQ flakes. (Closed)
Patch Set: Fix nit Created 4 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: appengine/findit/waterfall/flake/step_mapper.py
diff --git a/appengine/findit/waterfall/flake/step_mapper.py b/appengine/findit/waterfall/flake/step_mapper.py
index 38e49c52a34492642e4c4622bc0a62b2b8f11f91..ec151af75f4a3980781749967d302183c637a9ca 100644
--- a/appengine/findit/waterfall/flake/step_mapper.py
+++ b/appengine/findit/waterfall/flake/step_mapper.py
@@ -8,7 +8,7 @@ import os
import logging
from common import cache_decorator
-from common.http_client_appengine import HttpClientAppEngine as HttpClient
+from common.http_client_appengine import HttpClientAppengine as HttpClient
from waterfall import buildbot
from waterfall import swarming_util
@@ -68,7 +68,7 @@ def _GetMatchingWaterfallBuildStep(
logging.info(
'%s/%s/%s is not Swarmed yet.',
cq_build_step.master_name, cq_build_step.builder_name,
- cq_build_step.build_step)
+ cq_build_step.step_name)
return no_matching_result # Not on Swarm yet.
# Name of the step in the tags of a Swarming task.
@@ -104,7 +104,7 @@ def _GetMatchingWaterfallBuildStep(
logging.info(
'%s/%s/%s is mapped to %s/%s/%s',
cq_build_step.master_name, cq_build_step.builder_name,
- cq_build_step.build_step, wf_master_name, wf_builder_name,
+ cq_build_step.step_name, wf_master_name, wf_builder_name,
wf_step_name)
return wf_master_name, wf_builder_name, builds[0], wf_step_name

Powered by Google App Engine
This is Rietveld 408576698