Update review requests for bhyve.

[jhb]

Summary

• Explicitly mention the bhyve group on Phabricator.
• Request reviews of the userland components (libvmmapi, bhyve, bhyvectl, and bhyveload).

[araujo]

Sounds reasonable!

[imp]

Seems to achieve the desired policy goals.

[rgrimes]

This is good, but it is not clear on *who* can approve as bhyve, in the past that was Peter, is it now jhb & tychon, or anyone who just happens to join the bhyve group, which is not controller.

[araujo]

In D17848#381508, @rgrimes wrote:

This is good, but it is not clear on *who* can approve as bhyve, in the past that was Peter, is it now jhb & tychon, or anyone who just happens to join the bhyve group, which is not controller.

If the bhyve group cannot be controlled, in this case would be better add the team members name explicitly.
I thought the bhyve group were controlled.

[rgrimes]

In D17848#381511, @araujo wrote:

In D17848#381508, @rgrimes wrote:

This is good, but it is not clear on *who* can approve as bhyve, in the past that was Peter, is it now jhb & tychon, or anyone who just happens to join the bhyve group, which is not controller.

If the bhyve group cannot be controlled, in this case would be better add the team members name explicitly.
I thought the bhyve group were controlled.

It is controlled to the extent that only a committer can join it, but any committer who has joined it well get the bhyve umbrella accept check by default if they are a member and do an accept of a bhyve review. This could be changed. It is just how allanjude set it up when it was created.

[araujo]

After reflect a little bit more about what @rgrimes mentioned, how about we do something like this?

usr.sbin/bhyve* tychon, jhb, anish, araujo, rgrimes <...> Pre-commit review requested via #bhyve phabricator group.

It would applies also to libvmmapi and friends.

[jhb]

I think listing a lot of folks will start to get unreadable. I think we can start with the existing language and just use some common sense (if a couple of folks who aren't random committers approve userland changes, then that's probably fine). If we have to nail it down further in the future we can revisit it then.

[rgrimes]

I am good with the language as it is for now, lets get this commited sooner rather than later please.

[araujo]

I still feel locked out of usr.sbin/bhyve, mostly of the commits made there in the past year were made by myself or reviewed by myself. Would be fair have my name explicit in that area, I don't want bypass reviews, it is more about the effort I have put in that area and appreciation of it, otherwise doesn't make sense for me to keep contributing with bhyve.

I'm approving with disapproval.

[imp]

Would it make sense to add araujo's name to the usr.sbin/byhve line?