Issue 725166 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	lijeffrey@chromium.org
Closed:	May 2017
Cc:	█ robert...@chromium.org wylieb@chromium.org chanli@chromium.org st...@chromium.org lijeffrey@chromium.org liaoyuke@chromium.org
Components:	Tools>Test>FindIt>Flakiness
EstimatedDays:	----
NextAction:	----
OS:	----
Pri:	3
Type:	Bug

Blocked on:
issue 722555
issue 725132

[Findit] Pass lower/upper bound build numbers to recursive_flake_pipeline.py

Project Member Reported by lijeffrey@chromium.org, May 22 2017

Issue description

recursive_flake_pipeline.py only knows how to start at a build number and look back as needed. Pass in explicit lower/upper bounds to examine

Project Member

Comment 1 by bugdroid1@chromium.org, May 26 2017

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

commit a9a9a4e8ec55ac5864e6c38a92685eb697fad87a
Author: Jeffrey Li <lijeffrey@chromium.org>
Date: Fri May 26 00:37:25 2017

[Findit] Flake Analyzer - Adding upper/lower bounds to recursive_flake_pipeline

Bug:  725166 
Change-Id: I13ebc62108df71e1e84017b45a266ddd57779162
Reviewed-on: https://chromium-review.googlesource.com/514464
Reviewed-by: Chan Li <chanli@chromium.org>
Reviewed-by: Roberto Carrillo <robertocn@chromium.org>
Commit-Queue: Jeffrey Li <lijeffrey@chromium.org>

[modify] https://crrev.com/a9a9a4e8ec55ac5864e6c38a92685eb697fad87a/appengine/findit/waterfall/flake/test/recursive_flake_pipeline_test.py
[modify] https://crrev.com/a9a9a4e8ec55ac5864e6c38a92685eb697fad87a/appengine/findit/waterfall/flake/initialize_flake_pipeline.py
[modify] https://crrev.com/a9a9a4e8ec55ac5864e6c38a92685eb697fad87a/appengine/findit/waterfall/flake/recursive_flake_pipeline.py

Comment 2 by lijeffrey@chromium.org, May 30 2017

Status: Fixed (was: Assigned)

►

Issue 725166 link

Issue metadata

[Findit] Pass lower/upper bound build numbers to recursive_flake_pipeline.py

Issue description

Comment 1 by bugdroid1@chromium.org, May 26 2017

Comment 1 Deleted

Comment 2 by lijeffrey@chromium.org, May 30 2017

Comment 2 Deleted

Sign in to add a comment