Build delegation for CQ jobs

This CL allows test bots to delegate CQ build to faster builder bots instead of
compiling locally.

Similar to Issue 2061893003: Build delegation for perf tryjobs .

BUG= http://crbug.com/601699

Committed: https://chromium.googlesource.com/chromium/tools/build/+/f20aae69a5e30a8b3425009b5000be11c8699b92

[Ziqi Xiong, 2016-06-29 19:50:57]

Description was changed from

==========
This CL allows test bots to delegate build to faster builder bots instead of
compiling locally.
==========

to

==========
This CL allows test bots to delegate build to faster builder bots instead of
compiling locally.
==========

[Ziqi Xiong, 2016-06-29 19:50:58]

zxiong@google.com changed reviewers:
+ robertocn@chromium.org

[Ziqi Xiong, 2016-06-29 20:24:27]

Description was changed from

==========
This CL allows test bots to delegate build to faster builder bots instead of
compiling locally.
==========

to

==========
This CL allows test bots to delegate CQ build to faster builder bots instead of
compiling locally.
==========

[Ziqi Xiong, 2016-07-12 00:28:15]

Description was changed from

==========
This CL allows test bots to delegate CQ build to faster builder bots instead of
compiling locally.
==========

to

==========
This CL allows test bots to delegate CQ build to faster builder bots instead of
compiling locally.

Similar to Issue 2061893003: Build delegation for perf tryjobs .

BUG= http://crbug.com/601699
==========

[Ziqi Xiong, 2016-07-12 00:29:04]

Hi Roberto,

I think this CQ job CL is ready to commit. Let me know if you catch any bugs.

[sullivan, 2016-07-12 14:52:59]

sullivan@chromium.org changed reviewers:
+ dtu@chromium.org, sullivan@chromium.org

[RobertoCN, 2016-07-13 23:42:00]

This looks good. If the _compile method is no longer used then we should remove
it.

https://codereview.chromium.org/2104143003/diff/100001/scripts/slave/recipe_m...
File scripts/slave/recipe_modules/perf_try/api.py (right):

https://codereview.chromium.org/2104143003/diff/100001/scripts/slave/recipe_m...
scripts/slave/recipe_modules/perf_try/api.py:178: def _compile(self, name,
mastername, buildername, update_step, bot_db): # pragma: no cover
Should we remove this method? cq was the last user of it, right?

[Ziqi Xiong, 2016-07-14 00:15:33]

https://codereview.chromium.org/2104143003/diff/100001/scripts/slave/recipe_m...
File scripts/slave/recipe_modules/perf_try/api.py (right):

https://codereview.chromium.org/2104143003/diff/100001/scripts/slave/recipe_m...
scripts/slave/recipe_modules/perf_try/api.py:178: def _compile(self, name,
mastername, buildername, update_step, bot_db): # pragma: no cover
On 2016/07/13 23:42:00, RobertoCN wrote:
> Should we remove this method? cq was the last user of it, right?

Done.

[Ziqi Xiong, 2016-07-14 00:37:34]

The CQ bit was checked by zxiong@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-14 00:37:36]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-14 00:37:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-14 00:37:38]

Dry run: No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-infra-committers".
Note that this has nothing to do with OWNERS files.

[RobertoCN, 2016-07-14 22:12:54]

https://codereview.chromium.org/2104143003/diff/120001/scripts/slave/recipe_m...
File scripts/slave/recipe_modules/perf_try/api.py (right):

https://codereview.chromium.org/2104143003/diff/120001/scripts/slave/recipe_m...
scripts/slave/recipe_modules/perf_try/api.py:179: # We've had some cases where a
stale build directory was used on perf
This removal is incomplete. probably a rebase artifact?

[Ziqi Xiong, 2016-07-14 22:44:00]

https://codereview.chromium.org/2104143003/diff/120001/scripts/slave/recipe_m...
File scripts/slave/recipe_modules/perf_try/api.py (right):

https://codereview.chromium.org/2104143003/diff/120001/scripts/slave/recipe_m...
scripts/slave/recipe_modules/perf_try/api.py:179: # We've had some cases where a
stale build directory was used on perf
On 2016/07/14 22:12:54, RobertoCN wrote:
> This removal is incomplete. probably a rebase artifact?

Done.

[RobertoCN, 2016-07-15 23:00:42]

Before landing this, please explain in a doc how will we test that this patch
doesn't break cq jobs, and how will we know if we need to revert the patch. lgtm

[dtu, 2016-07-18 19:23:35]

lgtm

[Ziqi Xiong, 2016-07-18 19:56:43]

On 2016/07/15 23:00:42, RobertoCN wrote:
> Before landing this, please explain in a doc how will we test that this patch
> doesn't break cq jobs, and how will we know if we need to revert the patch.
lgtm

https://docs.google.com/a/google.com/document/d/1YNwuoNsCAyI_iN_1P-gDd_RxvFz2...

[Ziqi Xiong, 2016-07-18 19:58:16]

The CQ bit was checked by zxiong@google.com

[commit-bot: I haz the power, 2016-07-18 19:58:20]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-18 20:01:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-18 20:01:10]

Try jobs failed on following builders:
  Build Presubmit on master.tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/build...)

[Ziqi Xiong, 2016-07-18 20:11:58]

The CQ bit was checked by zxiong@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-18 20:12:04]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Ziqi Xiong, 2016-07-18 20:12:27]

The CQ bit was checked by zxiong@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-18 20:12:34]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-18 20:17:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-18 20:17:25]

Dry run: This issue passed the CQ dry run.

[Ziqi Xiong, 2016-07-18 20:19:36]

The CQ bit was checked by zxiong@google.com

[Ziqi Xiong, 2016-07-18 20:19:37]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertocn@chromium.org, dtu@chromium.org
Link to the patchset: https://codereview.chromium.org/2104143003/#ps160001
(title: "Rebase against master")

[commit-bot: I haz the power, 2016-07-18 20:19:48]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-18 20:24:53]

Description was changed from

==========
This CL allows test bots to delegate CQ build to faster builder bots instead of
compiling locally.

Similar to Issue 2061893003: Build delegation for perf tryjobs .

BUG= http://crbug.com/601699
==========

to

==========
This CL allows test bots to delegate CQ build to faster builder bots instead of
compiling locally.

Similar to Issue 2061893003: Build delegation for perf tryjobs .

BUG= http://crbug.com/601699

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/f20aae69a5e30a8b3425...
==========

[commit-bot: I haz the power, 2016-07-18 20:24:54]

Committed patchset #9 (id:160001) as
https://chromium.googlesource.com/chromium/tools/build/+/f20aae69a5e30a8b3425...