Analyze should use delta between patch sets (retry only failing steps/tests) |
||||||
Issue description1. Upload patch set 1, run bots. 2. See everything pass except one test step. 3. Upload patch set 2 to fix step, which modifies small part of CL. 4. See CQ run every test step again, instead of only the ones that failed in step 2 + the ones affected by the delta from patch set 1 and patch set 2. This could significantly speed up the commit queue and reduce load.
,
May 19 2017
,
May 22 2017
That's actually a non-trivial feature because it's nowhere trivial to predict what should be affected by delta you mentioned in light of potentially changing master with passage of time. This is currently not on our roadmap because it requires a good design doc and a person owning this through several quarters.
,
May 23 2017
See https://docs.google.com/a/google.com/document/d/1i0OxqLm6BpSrDge6EXjjNgBtVE0vxEfRlbCcZuNq_Ac/edit?usp=sharing for a doc I wrote about this in 2015. Agreed about benefits, and yeah - it'd be great to make that happen.
,
May 23 2017
It would be a good first pass to ignore master and just include the diff between the master at which each patchset was run when running analyze. That would be a huge improvement over run everything.
,
May 24 2018
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 25 2018
Still potentially useful. But still not on our road-map. Back into our backlog it goes. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by danakj@chromium.org
, May 19 2017