Moved committer list to chrome-infra-auth and deleted it from the repo

Moved committer list to chrome-infra-auth and deleted it from the repo

R=rmistry@chromium.org
BUG=chromium:511311
Committed: https://skia.googlesource.com/skia/+/613f20aa4b93c9196680608531b5496ec97dd090

[rmistry, 2015-09-02 11:13:16]

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

[rmistry, 2015-09-02 11:13:17]

https://codereview.chromium.org/1314153006/diff/1/CQ_COMMITTERS
File CQ_COMMITTERS (right):

https://codereview.chromium.org/1314153006/diff/1/CQ_COMMITTERS#newcode2
CQ_COMMITTERS:2:
https://chrome-infra-auth.appspot.com/auth/groups#project-skia-committers.
This messaging is very helpful. Thanks!

I just noticed that I am not a member of project-skia-admins and cannot
add/remove committers. Could you please add me there?

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

https://codereview.chromium.org/1314153006/diff/1/infra/branch-config/cq.cfg#...
infra/branch-config/cq.cfg:20:
"https://skia.googlesource.com/skia/+/master/CQ_COMMITTERS"
Could you please remove this line. The above line can end with
"from a\nfull Skia committer."
Heather and I will try to come up with a better link later.

[Sergiy Byelozyorov, 2015-09-02 17:31:17]

https://codereview.chromium.org/1314153006/diff/1/CQ_COMMITTERS
File CQ_COMMITTERS (right):

https://codereview.chromium.org/1314153006/diff/1/CQ_COMMITTERS#newcode2
CQ_COMMITTERS:2:
https://chrome-infra-auth.appspot.com/auth/groups#project-skia-committers.
On 2015/09/02 11:13:17, rmistry wrote:
> This messaging is very helpful. Thanks!
> 
> I just noticed that I am not a member of project-skia-admins and cannot
> add/remove committers. Could you please add me there?

Done.

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

https://codereview.chromium.org/1314153006/diff/1/infra/branch-config/cq.cfg#...
infra/branch-config/cq.cfg:20:
"https://skia.googlesource.com/skia/+/master/CQ_COMMITTERS"
On 2015/09/02 11:13:17, rmistry wrote:
> Could you please remove this line. The above line can end with
> "from a\nfull Skia committer."
> Heather and I will try to come up with a better link later.

Done.

[rmistry, 2015-09-02 18:53:04]

On 2015/09/02 17:31:17, Sergiy Byelozyorov wrote:
> https://codereview.chromium.org/1314153006/diff/1/CQ_COMMITTERS
> File CQ_COMMITTERS (right):
> 
> https://codereview.chromium.org/1314153006/diff/1/CQ_COMMITTERS#newcode2
> CQ_COMMITTERS:2:
> https://chrome-infra-auth.appspot.com/auth/groups#project-skia-committers.
> On 2015/09/02 11:13:17, rmistry wrote:
> > This messaging is very helpful. Thanks!
> > 
> > I just noticed that I am not a member of project-skia-admins and cannot
> > add/remove committers. Could you please add me there?
> 
> Done.
> 
> https://codereview.chromium.org/1314153006/diff/1/infra/branch-config/cq.cfg
> File infra/branch-config/cq.cfg (right):
> 
>
https://codereview.chromium.org/1314153006/diff/1/infra/branch-config/cq.cfg#...
> infra/branch-config/cq.cfg:20:
> "https://skia.googlesource.com/skia/+/master/CQ_COMMITTERS"
> On 2015/09/02 11:13:17, rmistry wrote:
> > Could you please remove this line. The above line can end with
> > "from a\nfull Skia committer."
> > Heather and I will try to come up with a better link later.
> 
> Done.

This LGTM.
We should probably wait for the config change discussed in the other thread to
be resolved before submitting this. Otherwise luci-config will not be able to
read cq.cfg and the existing CQ_COMMITTERS file will be empty causing the
current CQ to get an empty list.

[Sergiy Byelozyorov, 2015-09-02 20:32:38]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2015-09-02 20:32:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1314153006/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1314153006/40001

[Sergiy Byelozyorov, 2015-09-02 20:32:59]

The CQ bit was unchecked by sergiyb@chromium.org

[Sergiy Byelozyorov, 2015-09-02 20:33:36]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2015-09-02 20:33:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1314153006/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1314153006/40001

[rmistry, 2015-09-02 20:36:28]

On 2015/09/02 18:53:04, rmistry wrote:
> On 2015/09/02 17:31:17, Sergiy Byelozyorov wrote:
> > https://codereview.chromium.org/1314153006/diff/1/CQ_COMMITTERS
> > File CQ_COMMITTERS (right):
> > 
> > https://codereview.chromium.org/1314153006/diff/1/CQ_COMMITTERS#newcode2
> > CQ_COMMITTERS:2:
> > https://chrome-infra-auth.appspot.com/auth/groups#project-skia-committers.
> > On 2015/09/02 11:13:17, rmistry wrote:
> > > This messaging is very helpful. Thanks!
> > > 
> > > I just noticed that I am not a member of project-skia-admins and cannot
> > > add/remove committers. Could you please add me there?
> > 
> > Done.
> > 
> > https://codereview.chromium.org/1314153006/diff/1/infra/branch-config/cq.cfg
> > File infra/branch-config/cq.cfg (right):
> > 
> >
>
https://codereview.chromium.org/1314153006/diff/1/infra/branch-config/cq.cfg#...
> > infra/branch-config/cq.cfg:20:
> > "https://skia.googlesource.com/skia/+/master/CQ_COMMITTERS"
> > On 2015/09/02 11:13:17, rmistry wrote:
> > > Could you please remove this line. The above line can end with
> > > "from a\nfull Skia committer."
> > > Heather and I will try to come up with a better link later.
> > 
> > Done.
> 
> This LGTM.
> We should probably wait for the config change discussed in the other thread to
> be resolved before submitting this. Otherwise luci-config will not be able to
> read cq.cfg and the existing CQ_COMMITTERS file will be empty causing the
> current CQ to get an empty list.

I am assuming the issue mentioned above is resolved.

[commit-bot: I haz the power, 2015-09-02 20:37:56]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/613f20aa4b93c9196680608531b5496ec97dd090

[Sergiy Byelozyorov, 2015-09-02 20:39:34]

On 2015/09/02 18:53:04, rmistry wrote:
> We should probably wait for the config change discussed in the other thread to
> be resolved before submitting this. Otherwise luci-config will not be able to
> read cq.cfg and the existing CQ_COMMITTERS file will be empty causing the
> current CQ to get an empty list.

This should be already fixed because I've deployed a new version of the app.