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

Side by Side Diff: appengine/findit/waterfall/flake/trigger_flake_analyses_pipeline.py

Issue 2456033002: [Findit] Using flake_analysis_service and add bug number to check flake UI (Closed)
Patch Set: Addressing Comments 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 unified diff | Download patch
OLDNEW
1 # Copyright 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import logging 5 import logging
6 6
7 from common import time_util 7 from common import time_util
8 from common.pipeline_wrapper import BasePipeline 8 from common.pipeline_wrapper import BasePipeline
9 from model.flake. flake_analysis_request import FlakeAnalysisRequest 9 from model.flake.flake_analysis_request import FlakeAnalysisRequest
10 from model.wf_analysis import WfAnalysis 10 from model.wf_analysis import WfAnalysis
11 from model.wf_swarming_task import WfSwarmingTask 11 from model.wf_swarming_task import WfSwarmingTask
12 from waterfall.flake import flake_analysis_service 12 from waterfall.flake import flake_analysis_service
13 from waterfall.flake import triggering_sources 13 from waterfall.flake import triggering_sources
14 14
15 15
16 class TriggerFlakeAnalysesPipeline(BasePipeline): 16 class TriggerFlakeAnalysesPipeline(BasePipeline):
17 """A pipeline that automatically triggers flake analyses.""" 17 """A pipeline that automatically triggers flake analyses."""
18 18
19 # Arguments number differs from overridden method - pylint: disable=W0221 19 # Arguments number differs from overridden method - pylint: disable=W0221
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
55 master_name, builder_name, build_number, step, 55 master_name, builder_name, build_number, step,
56 time_util.GetUTCNow()) 56 time_util.GetUTCNow())
57 scheduled = flake_analysis_service.ScheduleAnalysisForFlake( 57 scheduled = flake_analysis_service.ScheduleAnalysisForFlake(
58 request, 'findit-for-me@appspot.gserviceaccount.com', False, 58 request, 'findit-for-me@appspot.gserviceaccount.com', False,
59 triggering_sources.FINDIT_PIPELINE) 59 triggering_sources.FINDIT_PIPELINE)
60 60
61 if scheduled: # pragma: no branch 61 if scheduled: # pragma: no branch
62 logging.info('%s/%s/%s has %s flaky tests.', 62 logging.info('%s/%s/%s has %s flaky tests.',
63 master_name, builder_name, build_number, len(flaky_tests)) 63 master_name, builder_name, build_number, len(flaky_tests))
64 logging.info('A flake analysis has been triggered for %s', test_name) 64 logging.info('A flake analysis has been triggered for %s', test_name)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698