Remove inactive accounts from OWNERS: armansito, keybuk, rpaquay

Remove inactive accounts from OWNERS: armansito, keybuk, rpaquay

These individuals are no longer performing reviews or submitting code
in the Chromium repository. Being in the OWNERS files confuses tools
such as git cl owners.

Committed: https://crrev.com/4558d7efd9ce7eb7bd04cce5a442baabc5d4b24b
Cr-Commit-Position: refs/heads/master@{#398938}

[scheib, 2016-06-02 00:13:39]

rockot:
  chrome/browser/extensions/api/bluetooth_low_energy/OWNERS
  device/OWNERS
  device/nfc/OWNERS
  device/usb/OWNERS
  extensions/browser/api/bluetooth/OWNERS
  extensions/browser/api/bluetooth_socket/OWNERS
  extensions/browser/api/hid/OWNERS
  extensions/browser/api/serial/OWNERS
  extensions/browser/api/socket/OWNERS
  extensions/browser/api/sockets_tcp/OWNERS
  extensions/browser/api/sockets_tcp_server/OWNERS
  extensions/browser/api/sockets_udp/OWNERS
  extensions/browser/api/system_network/OWNERS
  extensions/browser/api/usb/OWNERS
stevenjb:
  chrome/browser/chromeos/dbus/OWNERS
  chrome/browser/ui/webui/options/chromeos/OWNERS
  chromeos/dbus/OWNERS
  dbus/OWNERS

[scheib, 2016-06-03 17:15:15]

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

[commit-bot: I haz the power, 2016-06-03 17:15:41]

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

[scheib, 2016-06-03 17:16:00]

scheib@chromium.org changed reviewers:
+ rockot@chromium.org, stevenjb@chromium.org

[scheib, 2016-06-03 17:16:00]

rockot:
  chrome/browser/extensions/api/bluetooth_low_energy/OWNERS
  device/OWNERS
  device/nfc/OWNERS
  device/usb/OWNERS
  extensions/browser/api/bluetooth/OWNERS
  extensions/browser/api/bluetooth_socket/OWNERS
  extensions/browser/api/hid/OWNERS
  extensions/browser/api/serial/OWNERS
  extensions/browser/api/socket/OWNERS
  extensions/browser/api/sockets_tcp/OWNERS
  extensions/browser/api/sockets_tcp_server/OWNERS
  extensions/browser/api/sockets_udp/OWNERS
  extensions/browser/api/system_network/OWNERS
  extensions/browser/api/usb/OWNERS
stevenjb:
  chrome/browser/chromeos/dbus/OWNERS
  chrome/browser/ui/webui/options/chromeos/OWNERS
  chromeos/dbus/OWNERS
  dbus/OWNERS

[Ken Rockot(use gerrit already), 2016-06-03 17:19:05]

lgtm

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

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

[commit-bot: I haz the power, 2016-06-03 18:07:01]

Dry run: This issue passed the CQ dry run.

[stevenjb, 2016-06-08 01:09:11]

Please pick *someone* to own the extension APIs; the top level owners are not
going to be able to help much.

https://codereview.chromium.org/2030783002/diff/1/extensions/browser/api/blue...
File extensions/browser/api/bluetooth_socket/OWNERS (left):

https://codereview.chromium.org/2030783002/diff/1/extensions/browser/api/blue...
extensions/browser/api/bluetooth_socket/OWNERS:2: keybuk@chromium.org
Add me to this I guess, someone really needs to own it.

https://codereview.chromium.org/2030783002/diff/1/extensions/browser/api/sock...
File extensions/browser/api/sockets_tcp/OWNERS (left):

https://codereview.chromium.org/2030783002/diff/1/extensions/browser/api/sock...
extensions/browser/api/sockets_tcp/OWNERS:1: rpaquay@chromium.org
This really needs an owner, but I don't know anything about it.

https://codereview.chromium.org/2030783002/diff/1/extensions/browser/api/sock...
File extensions/browser/api/sockets_tcp_server/OWNERS (left):

https://codereview.chromium.org/2030783002/diff/1/extensions/browser/api/sock...
extensions/browser/api/sockets_tcp_server/OWNERS:1: rpaquay@chromium.org
This also really needs an owner.

https://codereview.chromium.org/2030783002/diff/1/extensions/browser/api/sock...
File extensions/browser/api/sockets_udp/OWNERS (left):

https://codereview.chromium.org/2030783002/diff/1/extensions/browser/api/sock...
extensions/browser/api/sockets_udp/OWNERS:1: rpaquay@chromium.org
Needs an owner.

https://codereview.chromium.org/2030783002/diff/1/extensions/browser/api/syst...
File extensions/browser/api/system_network/OWNERS (left):

https://codereview.chromium.org/2030783002/diff/1/extensions/browser/api/syst...
extensions/browser/api/system_network/OWNERS:1: rpaquay@chromium.org
Needs an owner.

[scheib, 2016-06-08 20:52:27]

scheib@chromium.org changed reviewers:
+ rdevlin.cronin@chromium.org, xiyuan@chromium.org

[scheib, 2016-06-08 20:52:27]

xiyuan@chromium.org & rdevlin.cronin@chromium.org, you both own Chrome Apps and
Chrome Extensions respectively.

Some of these directories no longer have code owners who participate in Chromium
development. I do not know of any suitable replacement for these
socket/networking related APIs:
extensions/browser/api/bluetooth_socket/OWNERS
extensions/browser/api/sockets_tcp
extensions/browser/api/sockets_tcp_server
extensions/browser/api/sockets_udp
extensions/browser/api/system_network
(Note, they are all Chrome Apps only)

As we have no staff maintaining them at an individual level, I believe they
revert back to your teams' ownership.

What would you like the OWNERS files to look like?

[Devlin, 2016-06-08 23:03:14]

On 2016/06/08 20:52:27, scheib wrote:
> mailto:xiyuan@chromium.org & mailto:rdevlin.cronin@chromium.org, you both own
Chrome Apps and
> Chrome Extensions respectively.
> 
> Some of these directories no longer have code owners who participate in
Chromium
> development. I do not know of any suitable replacement for these
> socket/networking related APIs:
> extensions/browser/api/bluetooth_socket/OWNERS
> extensions/browser/api/sockets_tcp
> extensions/browser/api/sockets_tcp_server
> extensions/browser/api/sockets_udp
> extensions/browser/api/system_network
> (Note, they are all Chrome Apps only)
> 
> As we have no staff maintaining them at an individual level, I believe they
> revert back to your teams' ownership.
> 
> What would you like the OWNERS files to look like?

Unless anyone has an idea of who specifically could own these (I don't), the
best choice I see is passing ownership to the apps team, assuming they intend to
maintain them as part of the apps platform.

[stevenjb, 2016-06-08 23:05:45]

I don't have a better solution. sgtm.


On Wed, Jun 8, 2016 at 4:03 PM, <rdevlin.cronin@chromium.org> wrote:

> On 2016/06/08 20:52:27, scheib wrote:
> > mailto:xiyuan@chromium.org & mailto:rdevlin.cronin@chromium.org, you
> both own
> Chrome Apps and
> > Chrome Extensions respectively.
> >
> > Some of these directories no longer have code owners who participate in
> Chromium
> > development. I do not know of any suitable replacement for these
> > socket/networking related APIs:
> > extensions/browser/api/bluetooth_socket/OWNERS
> > extensions/browser/api/sockets_tcp
> > extensions/browser/api/sockets_tcp_server
> > extensions/browser/api/sockets_udp
> > extensions/browser/api/system_network
> > (Note, they are all Chrome Apps only)
> >
> > As we have no staff maintaining them at an individual level, I believe
> they
> > revert back to your teams' ownership.
> >
> > What would you like the OWNERS files to look like?
>
> Unless anyone has an idea of who specifically could own these (I don't),
> the
> best choice I see is passing ownership to the apps team, assuming they
> intend to
> maintain them as part of the apps platform.
>
> https://codereview.chromium.org/2030783002/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[scheib, 2016-06-09 04:02:06]

Apps APIs now use apps/OWNERS.

[Devlin, 2016-06-09 17:05:58]

lgtm

[stevenjb, 2016-06-09 17:08:32]

On 2016/06/09 17:05:58, Devlin wrote:
> lgtm

Thanks! lgtm

[scheib, 2016-06-09 17:38:06]

The CQ bit was checked by scheib@chromium.org

[scheib, 2016-06-09 17:38:06]

The patchset sent to the CQ was uploaded after l-g-t-m from rockot@chromium.org
Link to the patchset: https://codereview.chromium.org/2030783002/#ps20001
(title: "use apps owners")

[commit-bot: I haz the power, 2016-06-09 17:38:20]

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

[commit-bot: I haz the power, 2016-06-09 17:45:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-09 17:46:27]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-09 17:49:14]

Description was changed from

==========
Remove inactive accounts from OWNERS: armansito, keybuk, rpaquay

These individuals are no longer performing reviews or submitting code
in the Chromium repository. Being in the OWNERS files confuses tools
such as git cl owners.
==========

to

==========
Remove inactive accounts from OWNERS: armansito, keybuk, rpaquay

These individuals are no longer performing reviews or submitting code
in the Chromium repository. Being in the OWNERS files confuses tools
such as git cl owners.

Committed: https://crrev.com/4558d7efd9ce7eb7bd04cce5a442baabc5d4b24b
Cr-Commit-Position: refs/heads/master@{#398938}
==========

[commit-bot: I haz the power, 2016-06-09 17:49:15]

Patchset 2 (id:??) landed as
https://crrev.com/4558d7efd9ce7eb7bd04cce5a442baabc5d4b24b
Cr-Commit-Position: refs/heads/master@{#398938}