Added CQ config to skia repo

Added CQ config to skia repo

This must land in sync with https://chromereviews.googleplex.com/204307013 and
revert in sync.

R=rmistry@chromium.org, tandrii@chromium.org
NOTRY=true
NOTREECHECKS=true
BUG=chromium:471767
Committed: https://skia.googlesource.com/skia/+/2baef5a1e104fe635c5b71fcdae159ba171c662c

[Sergiy Byelozyorov, 2015-06-05 12:50:16]

sergiyb@chromium.org changed reviewers:
+ tandrii@chromium.org

[tandrii(chromium), 2015-06-05 16:19:25]

lgtm

[rmistry, 2015-06-05 16:31:52]

rmistry@google.com changed required reviewers:
+ rmistry@chromium.org

[rmistry, 2015-06-08 11:39:15]

rmistry@google.com changed reviewers:
+ rmistry@google.com

[rmistry, 2015-06-08 11:39:15]

There is already a project-config/ dir that nodir had created, can we re-use
that one? it will be confusing to have both project-config and config.

https://codereview.chromium.org/1153093004/diff/1/infra/config/OWNERS
File infra/config/OWNERS (right):

https://codereview.chromium.org/1153093004/diff/1/infra/config/OWNERS#newcode1
infra/config/OWNERS:1: set noparent
Could you remove noparent here and instead add a noparent to the OWNERS in the
parent dir? (infra/OWNERS).
Other option is deleting this file and adding noparent and your name to
infra/OWNERS.

https://codereview.chromium.org/1153093004/diff/1/infra/config/cq.cfg
File infra/config/cq.cfg (right):

https://codereview.chromium.org/1153093004/diff/1/infra/config/cq.cfg#newcode4
infra/config/cq.cfg:4: version: 1
I am assuming the content here is identical to the old config file.

[Sergiy Byelozyorov, 2015-06-08 11:45:26]

On 2015/06/08 11:39:15, rmistry wrote:
> There is already a project-config/ dir that nodir had created, can we re-use
> that one? it will be confusing to have both project-config and config.

Yes, the different is that project-config contains project-wide configuration,
while config folder contains branch-specific configuration. A project may have
multiple CQs (one per branch), thus CQ config is stored in a branch-config. If
you prefer, we can either move the project-config to the infra/config branch
(where it will probably not be seen by most of the developers) or I can add an
explanation to the README.md in the infra dir.

https://codereview.chromium.org/1153093004/diff/1/infra/config/OWNERS
File infra/config/OWNERS (right):

https://codereview.chromium.org/1153093004/diff/1/infra/config/OWNERS#newcode1
infra/config/OWNERS:1: set noparent
On 2015/06/08 11:39:15, rmistry wrote:
> Could you remove noparent here and instead add a noparent to the OWNERS in the
> parent dir? (infra/OWNERS).
> Other option is deleting this file and adding noparent and your name to
> infra/OWNERS.

Done. I've moved myself and set-noparent to the parent OWNERS.

https://codereview.chromium.org/1153093004/diff/1/infra/config/cq.cfg
File infra/config/cq.cfg (right):

https://codereview.chromium.org/1153093004/diff/1/infra/config/cq.cfg#newcode4
infra/config/cq.cfg:4: version: 1
On 2015/06/08 11:39:15, rmistry wrote:
> I am assuming the content here is identical to the old config file.

Yes, I've tried my best not to make any mistakes when converting it, but please
check it again to avoid missing coverage.

[rmistry, 2015-06-08 11:51:48]

On 2015/06/08 11:45:26, Sergiy Byelozyorov wrote:
> On 2015/06/08 11:39:15, rmistry wrote:
> > There is already a project-config/ dir that nodir had created, can we re-use
> > that one? it will be confusing to have both project-config and config.
> 
> Yes, the different is that project-config contains project-wide configuration,
> while config folder contains branch-specific configuration. A project may have
> multiple CQs (one per branch), thus CQ config is stored in a branch-config. If
> you prefer, we can either move the project-config to the infra/config branch
> (where it will probably not be seen by most of the developers) or I can add an
> explanation to the README.md in the infra dir.

In that case should it be called branch-config ? that would be clearer to me.

> 
> https://codereview.chromium.org/1153093004/diff/1/infra/config/OWNERS
> File infra/config/OWNERS (right):
> 
> https://codereview.chromium.org/1153093004/diff/1/infra/config/OWNERS#newcode1
> infra/config/OWNERS:1: set noparent
> On 2015/06/08 11:39:15, rmistry wrote:
> > Could you remove noparent here and instead add a noparent to the OWNERS in
the
> > parent dir? (infra/OWNERS).
> > Other option is deleting this file and adding noparent and your name to
> > infra/OWNERS.
> 
> Done. I've moved myself and set-noparent to the parent OWNERS.
> 
> https://codereview.chromium.org/1153093004/diff/1/infra/config/cq.cfg
> File infra/config/cq.cfg (right):
> 
> https://codereview.chromium.org/1153093004/diff/1/infra/config/cq.cfg#newcode4
> infra/config/cq.cfg:4: version: 1
> On 2015/06/08 11:39:15, rmistry wrote:
> > I am assuming the content here is identical to the old config file.
> 
> Yes, I've tried my best not to make any mistakes when converting it, but
please
> check it again to avoid missing coverage.

[rmistry, 2015-06-08 12:03:00]

rmistry@google.com changed required reviewers:
+ rmistry@google.com
- rmistry@chromium.org

[rmistry, 2015-06-08 12:03:01]

LGTM

https://codereview.chromium.org/1153093004/diff/1/infra/config/cq.cfg
File infra/config/cq.cfg (right):

https://codereview.chromium.org/1153093004/diff/1/infra/config/cq.cfg#newcode4
infra/config/cq.cfg:4: version: 1
On 2015/06/08 11:45:26, Sergiy Byelozyorov wrote:
> On 2015/06/08 11:39:15, rmistry wrote:
> > I am assuming the content here is identical to the old config file.
> 
> Yes, I've tried my best not to make any mistakes when converting it, but
please
> check it again to avoid missing coverage.

The only one I see missing is:
"commit_user": "commit-bot@chromium.org"
That was probably intentionally removed.

[Sergiy Byelozyorov, 2015-06-08 12:08:32]

On 2015/06/08 12:03:01, rmistry wrote:
> LGTM
> 
> https://codereview.chromium.org/1153093004/diff/1/infra/config/cq.cfg
> File infra/config/cq.cfg (right):
> 
> https://codereview.chromium.org/1153093004/diff/1/infra/config/cq.cfg#newcode4
> infra/config/cq.cfg:4: version: 1
> On 2015/06/08 11:45:26, Sergiy Byelozyorov wrote:
> > On 2015/06/08 11:39:15, rmistry wrote:
> > > I am assuming the content here is identical to the old config file.
> > 
> > Yes, I've tried my best not to make any mistakes when converting it, but
> please
> > check it again to avoid missing coverage.
> 
> The only one I see missing is:
> "commit_user": mailto:"commit-bot@chromium.org"
> That was probably intentionally removed.

Yes it was removed because this is now default value.

[Sergiy Byelozyorov, 2015-06-08 12:09:07]

The CQ bit was checked by sergiyb@chromium.org

[Sergiy Byelozyorov, 2015-06-08 12:09:07]

The patchset sent to the CQ was uploaded after l-g-t-m from
tandrii@chromium.org, rmistry@google.com
Link to the patchset: https://codereview.chromium.org/1153093004/#ps140001
(title: "Added a comment to refs.cfg")

[commit-bot: I haz the power, 2015-06-08 12:09:14]

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

[Sergiy Byelozyorov, 2015-06-08 12:09:37]

The CQ bit was unchecked by sergiyb@chromium.org

[Sergiy Byelozyorov, 2015-06-08 12:09:50]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2015-06-08 12:10:03]

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

[Sergiy Byelozyorov, 2015-06-08 12:12:07]

The CQ bit was unchecked by sergiyb@chromium.org

[Sergiy Byelozyorov, 2015-06-08 12:12:18]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2015-06-08 12:12:24]

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

[commit-bot: I haz the power, 2015-06-08 12:13:01]

Committed patchset #8 (id:140001) as
https://skia.googlesource.com/skia/+/2baef5a1e104fe635c5b71fcdae159ba171c662c