build: add recipe tests to infra/infra_repo_trybot

build: add recipe tests to infra/infra_repo_trybot

BUG=617565
Committed: https://chromium.googlesource.com/chromium/tools/build/+/978385980717ba748aa3492de68429a18e5d1530

[Paweł Hajdan Jr., 2016-06-07 11:16:25]

phajdan.jr@chromium.org changed reviewers:
+ sergiyb@chromium.org, tandrii@chromium.org

[Paweł Hajdan Jr., 2016-06-07 11:16:29]

The CQ bit was checked by phajdan.jr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-07 11:16:39]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2042243002/1

[commit-bot: I haz the power, 2016-06-07 11:20:49]

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

[commit-bot: I haz the power, 2016-06-07 11:20:50]

Dry run: This issue passed the CQ dry run.

[Michael Achenbach, 2016-06-07 11:54:25]

machenbach@chromium.org changed reviewers:
+ machenbach@chromium.org

[Michael Achenbach, 2016-06-07 11:54:25]

lgtm

[Paweł Hajdan Jr., 2016-06-07 11:55:52]

The CQ bit was checked by phajdan.jr@chromium.org

[commit-bot: I haz the power, 2016-06-07 11:56:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2042243002/1

[commit-bot: I haz the power, 2016-06-07 12:00:00]

Description was changed from

==========
build: add recipe tests to infra/infra_repo_trybot

BUG=617565
==========

to

==========
build: add recipe tests to infra/infra_repo_trybot

BUG=617565

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

[commit-bot: I haz the power, 2016-06-07 12:00:01]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/build/+/978385980717ba748aa3...

[Sergiy Byelozyorov, 2016-06-07 12:01:57]

lgtm

[tandrii(chromium), 2016-06-07 12:04:58]

Please add reference in description to CL which moves the recipes to
infra/infra, and vise versa, so that in case someone later on needs to revert
the CLs, it's easier to figure out the dependencies. And no, just BUG= isn't
sufficient, though necessary.

[Paweł Hajdan Jr., 2016-06-07 12:23:32]

On 2016/06/07 at 12:04:58, tandrii wrote:
> Please add reference in description to CL which moves the recipes to
infra/infra, and vise versa, so that in case someone later on needs to revert
the CLs, it's easier to figure out the dependencies. And no, just BUG= isn't
sufficient, though necessary.

Okay - please note I don't always notice replies to closed CLs.

FWIW, the infra recipes CL is https://codereview.chromium.org/2041313002 .

[tandrii(chromium), 2016-06-07 12:40:35]

On 2016/06/07 12:23:32, Paweł Hajdan Jr. wrote:
> On 2016/06/07 at 12:04:58, tandrii wrote:
> > Please add reference in description to CL which moves the recipes to
> infra/infra, and vise versa, so that in case someone later on needs to revert
> the CLs, it's easier to figure out the dependencies. And no, just BUG= isn't
> sufficient, though necessary.
> 
> Okay - please note I don't always notice replies to closed CLs.
Noted.

> FWIW, the infra recipes CL is https://codereview.chromium.org/2041313002 .
Again, this should go to the description.