<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 7, 2019 at 1:24 AM Samuel Ortiz <<a href="mailto:samuel.ortiz@intel.com">samuel.ortiz@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">All,<br>
<br>
As discussed during last week's call, I'd like to propose a light<br>
process for PR reviews and gate keeping.<br>
<br>
At the moment, when a PR lands in any of the existing crates, no one<br>
gets assigned to review it and we're asking for 2 review approvals<br>
before merging it. Those approvals should come from members of the<br>
crate's "team", and we need to arbitrarily build those teams on the<br>
fly based on the crate activity and contributions.<br>
<br>
As the number of incoming PRs is increasing, I feel we're starting to<br>
see some limitations with the above described model. While many PRs get<br>
actively reviewed, others slightly less. Also, pinging people for<br>
reviews and approvals is handled manually at the moment.<br>
<br>
In order to improve that situation, here is my proposal:<br>
<br>
- Have a team of 8-10 rust-vmm gatekeepers. 2 randomly selected<br>
  gatekeepers will be automatically assigned to any incoming PR.<br>
- Any gate keeper is free to assign someone else from the gate keeper<br>
  team if she/he does not have the badwidth/knowledge to review the PR.<br>
- Encourage CODEOWNERS[1] file additions. Not all repos may need such<br>
  ownership so I don't think this should be mandatory.<br>
- Any PR will be mergeable when any of the below conditions are met:<br>
  * When a repo has a CODEOWNERS file, 1 code owner and 1 gate keeper<br>
    approved it.<br>
  * On CODEOWNERS-less repos, 2 gate keepers approved it.<br>
- Initial PRs for empty crates could be handled differently and<br>
  informally require more approvals from different people.<br>
<br>
I guess the natural question coming next is who should be part of this<br>
gate keeper team. rust-vmm being almost 6 months old, I think we now<br>
have a good idea of who are the most active contributors to the<br>
project. And I think it's logical to build this initial gate keepers<br>
team from those initial active contributors:<br>
<br>
Andreea Florescu   @andreeaflorescu<br>
Alexandru Agache   @alexandruag<br>
Alexandra Iordache @aghecenco<br>
Jiang Liu          @jiangliu<br>
Paolo Bonzini      @bonzini<br>
Zach Reizner       @zachreizner<br>
Jenny Mankin       @jennymankin<br>
Sebastien Boeuf    @sboeuf<br>
Rob Bradford       @rbradford<br>
Samuel Ortiz       @sameo<br></blockquote><div><span class="gmail_default" style="font-family:arial,helvetica,sans-serif">I agree to serve as a gatekeeper and the overall plan sounds good.</span> <span class="gmail_default" style="font-family:arial,helvetica,sans-serif">Thanks.</span></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Comments, disagreements, additions, etc are all very welcome.<br>
<br>
Cheers,<br>
Samuel.<br>
<br>
[1] <a href="https://help.github.com/en/articles/about-code-owners" rel="noreferrer" target="_blank">https://help.github.com/en/articles/about-code-owners</a><br>
---------------------------------------------------------------------<br>
Intel Corporation SAS (French simplified joint stock company)<br>
Registered headquarters: "Les Montalets"- 2, rue de Paris, <br>
92196 Meudon Cedex, France<br>
Registration Number:  302 456 199 R.C.S. NANTERRE<br>
Capital: 4,572,000 Euros<br>
<br>
This e-mail and any attachments may contain confidential material for<br>
the sole use of the intended recipient(s). Any review or distribution<br>
by others is strictly prohibited. If you are not the intended<br>
recipient, please contact the sender and delete all copies.<br>
<br>
<br>
_______________________________________________<br>
Rust-vmm mailing list<br>
<a href="mailto:Rust-vmm@lists.opendev.org" target="_blank">Rust-vmm@lists.opendev.org</a><br>
<a href="http://lists.opendev.org/cgi-bin/mailman/listinfo/rust-vmm" rel="noreferrer" target="_blank">http://lists.opendev.org/cgi-bin/mailman/listinfo/rust-vmm</a><br>
</blockquote></div></div>