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

Issue 819322 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Rate-limit and prioritize sets of CLs to test in CQ

Project Member Reported by pho...@chromium.org, Mar 6 2018

Issue description

Exonerating most CLs in a 50+ CL build doesn't make sense - the next run will also probably fail. It would be better to cap the # of CLs, rather than the number of builds processed by CL-Exonerator.
 
Cc: jclinton@chromium.org
I argue that it's better to exonerate all CLs at once so that developers don't keep manually reset flags, not realizing the system will do it later on.

It's much better to instead improve the PreCQ and CQ CL selection logic to limit the number of CL "sets" it will test at a time.

That approach should be easier for users to understand, and be more flexible, since it will also help with recovery build "build down" issues, and from unusually busy development days. We SHOULD generate omens any time there is a backlog of CLs to test, however.

We might need to put some thought into the "fairest" way for both the CQ and PreCQ to decide which CLs to test first if they are hitting their limits.
Summary: Rate-limit and prioritize sets of CLs to test in CQ (was: Rate-limit the number of CLs exonerated, not just the number of builds)
OK, so let's not rate-limit exonerator. Bug title changed.

I'd argue that a fair way to rate-limit CLs in the CQ would be to use CL-Scanner to select non-riksy subsets of the CLs. The whole reason we want to select a subset of CLs to begin with is to reduce the risk of the run failing, so it would make sense to take advantage of the CL risk signal to make that decision more intelligently.

Comment 3 by pho...@chromium.org, Apr 20 2018

Components: -Infra>Client>ChromeOS Infra>Client>ChromeOS>CI
Owner: jclinton@chromium.org
Status: Untriaged (was: Assigned)
Passing to jclinton@ to prioritize
Status: Assigned (was: Untriaged)
This bug has an owner, thus, it's been triaged. Changing status to "assigned".
Status: Untriaged (was: Assigned)
As it says above, no it hasn't been.
Owner: ----
Will be triaged during our triage process if it has no owner.

Sign in to add a comment