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

Issue 745002 link

Starred by 1 user

Issue metadata

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

Blocking:
issue 744711



Sign in to add a comment

[Findit] Flake Analyzer - Make the first RecursiveFlakePipeline the total root

Project Member Reported by wylieb@chromium.org, Jul 17 2017

Issue description

This will be useful when inspecting pipelines via the _ah viewer.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 19 2017

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

commit f28296da94b63752810ef8ab08904b8456cce503
Author: Brandon Wylie <wylieb@chromium.org>
Date: Wed Jul 19 20:16:55 2017

[Findit] Flake Analyzer - First RecursiveFlakePipeline is root.

Instead of calling start every time a new build number is identified, now
new RecursiveFlakePipeline is called with yield, making the first
RecursiveFlakePipeline call the root pipeline. This will make it easier to
inspect the whole pipeline flow.

Bug:  745002 
Change-Id: Iefcf244de4cf3510c1f4cc319bcd9784ec08a4cf
Reviewed-on: https://chromium-review.googlesource.com/576248
Reviewed-by: Jeffrey Li <lijeffrey@chromium.org>
Commit-Queue: Jeffrey Li <lijeffrey@chromium.org>

[modify] https://crrev.com/f28296da94b63752810ef8ab08904b8456cce503/appengine/findit/waterfall/flake/recursive_flake_pipeline.py

Comment 2 by wylieb@chromium.org, Jul 19 2017

Status: Assigned (was: Untriaged)

Comment 3 by wylieb@chromium.org, Jul 19 2017

Blocking: 744711

Comment 4 by wylieb@chromium.org, Jul 24 2017

Status: Fixed (was: Assigned)

Sign in to add a comment