Mark WebRTC project as public in luci-config

Mark WebRTC project as public in luci-config

NOTRY=true
R=kjellander@chromium.org, nodir@chromium.org
BUG=497093
Committed: https://crrev.com/d60a7993bf14c6090bc6df258feaf914a1ce46c5
Cr-Commit-Position: refs/heads/master@{#9609}

[Sergiy Byelozyorov, 2015-07-21 15:53:30]

sergiyb@chromium.org changed reviewers:
+ niklas.enbom@webrtc.org
- kjellander@chromium.org

[Sergiy Byelozyorov, 2015-07-21 15:53:30]

Niklas, can you please LGTM this infra-related change, since Henrik is OOO?

[Sergiy Byelozyorov, 2015-07-21 15:58:48]

On 2015/07/21 15:53:30, Sergiy Byelozyorov wrote:
> Niklas, can you please LGTM this infra-related change, since Henrik is OOO?

For context, this will make configs for WebRTC project publicly visible in
luci-config app (they are already publicly available at
https://chromium.googlesource.com/external/webrtc/+/master/infra/), which is
used by various services in infra and will soon be used by codereview.webrtc.org
to determine whether CQ checkbox/button should be enabled or not.

[niklas.enbom, 2015-07-21 16:06:27]

lgtm
On 2015/07/21 15:58:48, Sergiy Byelozyorov wrote:
> On 2015/07/21 15:53:30, Sergiy Byelozyorov wrote:
> > Niklas, can you please LGTM this infra-related change, since Henrik is OOO?
> 
> For context, this will make configs for WebRTC project publicly visible in
> luci-config app (they are already publicly available at
> https://chromium.googlesource.com/external/webrtc/+/master/infra/), which is
> used by various services in infra and will soon be used by
http://codereview.webrtc.org
> to determine whether CQ checkbox/button should be enabled or not.

[nodir, 2015-07-21 16:25:53]

How urgent is this change? it adds more work: I will have to send a CL to remove
these files from this branch because they will move to infra/config branch

[Sergiy Byelozyorov, 2015-07-21 22:09:09]

On 2015/07/21 16:25:53, nodir wrote:
> How urgent is this change? it adds more work: I will have to send a CL to
remove
> these files from this branch because they will move to infra/config branch

Wouldn't you just need to move the whole directory to infra/config branch? Note
that this directory already exists, so it doesn't really add more to your work.

[nodir, 2015-07-22 12:05:05]

On 2015/07/21 22:09:09, Sergiy Byelozyorov wrote:
> On 2015/07/21 16:25:53, nodir wrote:
> > How urgent is this change? it adds more work: I will have to send a CL to
> remove
> > these files from this branch because they will move to infra/config branch
> 
> Wouldn't you just need to move the whole directory to infra/config branch

Cross-branch repo move is delete and add CLs

> Note
> that this directory already exists, so it doesn't really add more to your
work.

You are right, lgtm

[Sergiy Byelozyorov, 2015-07-22 12:56:38]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2015-07-22 12:56:48]

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

[Sergiy Byelozyorov, 2015-07-22 13:06:05]

The CQ bit was unchecked by sergiyb@chromium.org

[Sergiy Byelozyorov, 2015-07-22 13:06:20]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2015-07-22 13:06:32]

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

[commit-bot: I haz the power, 2015-07-22 13:11:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-07-22 13:12:25]

Patchset 1 (id:??) landed as
https://crrev.com/d60a7993bf14c6090bc6df258feaf914a1ce46c5
Cr-Commit-Position: refs/heads/master@{#9609}