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

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

Issue 2026283002: [Findit] Adding logic to force try jobs regardless of bailout or previous results (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Addressing comments and rebase 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
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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 from datetime import datetime 5 from datetime import datetime
6 6
7 from common import appengine_util 7 from common import appengine_util
8 from common.pipeline_wrapper import BasePipeline 8 from common.pipeline_wrapper import BasePipeline
9 from model import analysis_status 9 from model import analysis_status
10 from model.wf_analysis import WfAnalysis 10 from model.wf_analysis import WfAnalysis
11 from waterfall.detect_first_failure_pipeline import DetectFirstFailurePipeline 11 from waterfall.detect_first_failure_pipeline import DetectFirstFailurePipeline
12 from waterfall.extract_deps_info_pipeline import ExtractDEPSInfoPipeline 12 from waterfall.extract_deps_info_pipeline import ExtractDEPSInfoPipeline
13 from waterfall.extract_signal_pipeline import ExtractSignalPipeline 13 from waterfall.extract_signal_pipeline import ExtractSignalPipeline
14 from waterfall.identify_culprit_pipeline import IdentifyCulpritPipeline 14 from waterfall.identify_culprit_pipeline import IdentifyCulpritPipeline
15 from waterfall.pull_changelog_pipeline import PullChangelogPipeline 15 from waterfall.pull_changelog_pipeline import PullChangelogPipeline
16 from waterfall.start_try_job_on_demand_pipeline import ( 16 from waterfall.start_try_job_on_demand_pipeline import (
17 StartTryJobOnDemandPipeline) 17 StartTryJobOnDemandPipeline)
18 18
19 19
20 class AnalyzeBuildFailurePipeline(BasePipeline): 20 class AnalyzeBuildFailurePipeline(BasePipeline):
21 21
22 def __init__(self, master_name, builder_name, build_number, build_completed): 22 def __init__(self, master_name, builder_name, build_number, build_completed,
23 force_rerun_try_job):
23 super(AnalyzeBuildFailurePipeline, self).__init__( 24 super(AnalyzeBuildFailurePipeline, self).__init__(
24 master_name, builder_name, build_number, build_completed) 25 master_name, builder_name, build_number, build_completed,
26 force_rerun_try_job)
25 self.master_name = master_name 27 self.master_name = master_name
26 self.builder_name = builder_name 28 self.builder_name = builder_name
27 self.build_number = build_number 29 self.build_number = build_number
28 30
29 def _LogUnexpectedAborting(self, was_aborted): 31 def _LogUnexpectedAborting(self, was_aborted):
30 """Marks the WfAnalysis status as error, indicating that it was aborted. 32 """Marks the WfAnalysis status as error, indicating that it was aborted.
31 33
32 Args: 34 Args:
33 was_aborted (bool): True if the pipeline was aborted, otherwise False. 35 was_aborted (bool): True if the pipeline was aborted, otherwise False.
34 """ 36 """
(...skipping 14 matching lines...) Expand all
49 analysis = WfAnalysis.Get(master_name, builder_name, build_number) 51 analysis = WfAnalysis.Get(master_name, builder_name, build_number)
50 analysis.pipeline_status_path = self.pipeline_status_path() 52 analysis.pipeline_status_path = self.pipeline_status_path()
51 analysis.status = analysis_status.RUNNING 53 analysis.status = analysis_status.RUNNING
52 analysis.result_status = None 54 analysis.result_status = None
53 analysis.start_time = datetime.utcnow() 55 analysis.start_time = datetime.utcnow()
54 analysis.version = appengine_util.GetCurrentVersion() 56 analysis.version = appengine_util.GetCurrentVersion()
55 analysis.end_time = None 57 analysis.end_time = None
56 analysis.put() 58 analysis.put()
57 59
58 # Arguments number differs from overridden method - pylint: disable=W0221 60 # Arguments number differs from overridden method - pylint: disable=W0221
59 def run(self, master_name, builder_name, build_number, build_completed): 61 def run(self, master_name, builder_name, build_number, build_completed,
62 force_rerun_try_job):
60 self._ResetAnalysis(master_name, builder_name, build_number) 63 self._ResetAnalysis(master_name, builder_name, build_number)
61 64
62 # The yield statements below return PipelineFutures, which allow subsequent 65 # The yield statements below return PipelineFutures, which allow subsequent
63 # pipelines to refer to previous output values. 66 # pipelines to refer to previous output values.
64 # https://github.com/GoogleCloudPlatform/appengine-pipelines/wiki/Python 67 # https://github.com/GoogleCloudPlatform/appengine-pipelines/wiki/Python
65 failure_info = yield DetectFirstFailurePipeline( 68 failure_info = yield DetectFirstFailurePipeline(
66 master_name, builder_name, build_number) 69 master_name, builder_name, build_number)
67 change_logs = yield PullChangelogPipeline(failure_info) 70 change_logs = yield PullChangelogPipeline(failure_info)
68 deps_info = yield ExtractDEPSInfoPipeline(failure_info, change_logs) 71 deps_info = yield ExtractDEPSInfoPipeline(failure_info, change_logs)
69 signals = yield ExtractSignalPipeline(failure_info) 72 signals = yield ExtractSignalPipeline(failure_info)
70 heuristic_result = yield IdentifyCulpritPipeline( 73 heuristic_result = yield IdentifyCulpritPipeline(
71 failure_info, change_logs, deps_info, signals, build_completed) 74 failure_info, change_logs, deps_info, signals, build_completed)
72 yield StartTryJobOnDemandPipeline( 75 yield StartTryJobOnDemandPipeline(
73 failure_info, signals, build_completed, heuristic_result) 76 failure_info, signals, build_completed, force_rerun_try_job,
77 heuristic_result)
OLDNEW
« no previous file with comments | « appengine/findit/templates/build_failure.html ('k') | appengine/findit/waterfall/build_failure_analysis_pipelines.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698