New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 757667 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 754527
issue 757661



Sign in to add a comment

[Findit] Merge pipelines

Project Member Reported by chanli@chromium.org, Aug 22 2017

Issue description

Candidate pipelines to merge: merge PullChangelogPipeline and ExtractDEPSInfoPipeline into IdentifyCulpritPipeline
 

Comment 1 by chanli@chromium.org, Aug 23 2017

Before merging pipelines, first we need to separate logic from pipelines.

Comment 4 by chanli@chromium.org, Aug 28 2017

Blocking: 757661
Project Member

Comment 5 by bugdroid1@chromium.org, Aug 29 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/96990e4e17768932817128ac955eb5754c5cdce3

commit 96990e4e17768932817128ac955eb5754c5cdce3
Author: Chan <chanli@chromium.org>
Date: Tue Aug 29 18:09:02 2017

[Findit] Merge pipelines.

Merge PullChangeLogPipeline and ExtractDepsInfoPipeline into IdentifyCulpritPipeline.

Bug:  757667 
Change-Id: I67705859144b23c3dbacd50eae1860a1ef15e51a
Reviewed-on: https://chromium-review.googlesource.com/636301
Commit-Queue: Chan Li <chanli@chromium.org>
Reviewed-by: Shuotao Gao <stgao@chromium.org>

[modify] https://crrev.com/96990e4e17768932817128ac955eb5754c5cdce3/appengine/findit/waterfall/test/build_failure_analysis_test.py
[delete] https://crrev.com/73df5c9cc0d0a66a7e84d3b97085e62ec1085f58/appengine/findit/waterfall/test/extract_deps_info_pipeline_test.py
[delete] https://crrev.com/73df5c9cc0d0a66a7e84d3b97085e62ec1085f58/appengine/findit/waterfall/test/pull_changelog_pipeline_test.py
[modify] https://crrev.com/96990e4e17768932817128ac955eb5754c5cdce3/appengine/findit/waterfall/test/identify_culprit_pipeline_test.py
[modify] https://crrev.com/96990e4e17768932817128ac955eb5754c5cdce3/appengine/findit/waterfall/test/analyze_build_failure_pipeline_test.py
[delete] https://crrev.com/73df5c9cc0d0a66a7e84d3b97085e62ec1085f58/appengine/findit/waterfall/extract_deps_info_pipeline.py
[delete] https://crrev.com/73df5c9cc0d0a66a7e84d3b97085e62ec1085f58/appengine/findit/waterfall/pull_changelog_pipeline.py
[modify] https://crrev.com/96990e4e17768932817128ac955eb5754c5cdce3/appengine/findit/waterfall/analyze_build_failure_pipeline.py
[modify] https://crrev.com/96990e4e17768932817128ac955eb5754c5cdce3/appengine/findit/waterfall/identify_culprit_pipeline.py
[modify] https://crrev.com/96990e4e17768932817128ac955eb5754c5cdce3/appengine/findit/waterfall/build_failure_analysis.py

Comment 6 by chanli@chromium.org, Aug 31 2017

Status: Fixed (was: Assigned)

Sign in to add a comment