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

Issue 735114 link

Starred by 1 user
Status: Fixed
Owner:
lijeffrey@chromium.org
Closed: Jun 2017
Cc:
█ robert...@chromium.org
wylieb@chromium.org
chanli@chromium.org
st...@chromium.org
lijeffrey@chromium.org
liaoyuke@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 734660



Sign in to add a comment

[Findit] Flake Analyzer - Move try job logic into separate pipeline

Project Member Reported by lijeffrey@chromium.org, Jun 20 2017 
recursive_flake_pipeline is responsible for determining what to call recursive_flake_try_job_pipeline with or to bail out. Instead, move the logic into a separate sub pipeline that handles single case, empty blame list, determining the first revision to run, etc.

Comment 1 by lijeffrey@chromium.org, Jun 23 2017

Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 28 2017 

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

commit 8921148e4a59539d533fb6192f15ed92f9c49d35
Author: Jeffrey Li <lijeffrey@chromium.org>
Date: Wed Jun 28 23:07:40 2017

[Findit] Flake Analyzer - Separating try jobs from recursive_flake_pipeline

Bug:  735114 
Change-Id: Ie3e8a13aed65b29d2491e727cf20ae6a6847dd1c
Reviewed-on: https://chromium-review.googlesource.com/549190
Commit-Queue: Jeffrey Li <lijeffrey@chromium.org>
Reviewed-by: Roberto Carrillo <robertocn@chromium.org>

[modify] https://crrev.com/8921148e4a59539d533fb6192f15ed92f9c49d35/appengine/findit/waterfall/flake/recursive_flake_pipeline.py
[add] https://crrev.com/8921148e4a59539d533fb6192f15ed92f9c49d35/appengine/findit/waterfall/flake/test/initialize_flake_try_job_pipeline_test.py
[modify] https://crrev.com/8921148e4a59539d533fb6192f15ed92f9c49d35/appengine/findit/model/flake/test/master_flake_analysis_test.py
[modify] https://crrev.com/8921148e4a59539d533fb6192f15ed92f9c49d35/appengine/findit/waterfall/flake/test/recursive_flake_pipeline_test.py
[modify] https://crrev.com/8921148e4a59539d533fb6192f15ed92f9c49d35/appengine/findit/model/flake/master_flake_analysis.py
[modify] https://crrev.com/8921148e4a59539d533fb6192f15ed92f9c49d35/appengine/findit/waterfall/flake/recursive_flake_try_job_pipeline.py
[add] https://crrev.com/8921148e4a59539d533fb6192f15ed92f9c49d35/appengine/findit/waterfall/flake/initialize_flake_try_job_pipeline.py
[modify] https://crrev.com/8921148e4a59539d533fb6192f15ed92f9c49d35/appengine/findit/waterfall/flake/test/recursive_flake_try_job_pipeline_test.py

Comment 3 by lijeffrey@chromium.org, Jun 30 2017

Status: Fixed (was: Started)
►

Sign in to add a comment