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

Issue 713508 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 705023
Owner: ----
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

throttled selection of CLs should take dependency changes into account

Project Member Reported by vapier@chromium.org, Apr 20 2017

Issue description

i've got two CLs that depend on each other:
https://chromium-review.googlesource.com/481200
https://chromium-review.googlesource.com/481201

because the tree is throttled, the CQ is only testing a random subset.  so far, it's managed to randomly pick just one, but then reject it at the end because it didn't pick up the other one.
https://uberchromegw.corp.google.com/i/chromeos/builders/master-paladin/builds/14349/steps/CommitQueueSync/logs/stdio/text

we should either include the dep tree of any CLs we picked at random, or we should (silently) add them to the filtered out list.
 
Cc: nxia@chromium.org

Comment 2 by nxia@chromium.org, Apr 24 2017

Mergedinto: 705023
Status: Duplicate (was: Available)

Sign in to add a comment