Tao, Rico - thank you! for opting in I have taken our current internal list of 'volunteers', modified it a bit and added the new volunteers in, and then embellished it with some useful other info. I've placed it up on the wiki. It can probably do with some shuffling of the rota - but we can work on that. https://github.com/kata-containers/community/wiki/Review-Team-Rota Can I ask all members in the Rota table to check they are available on those dates please. JamesH and I threw around some funny names between us. Taking a bit of inspiration from JamesH and something EricE posted, I particularly like the idea of the 'Kat herding team', but I settled on the more sedate 'Kata review team'. Still, if enough folks want to change that name .... ;-) We should probably make a top level wiki page with a link to that page (otherwise finding wiki pages on github can be a bit of an art), and probably make a PR to add a link into the community docs as well... once the page is settled then I'll try to remember to do that (and will go open an Issue for it right now..) Graham
-----Original Message----- From: Tao Peng [mailto:bergwolf@hyper.sh] Sent: Tuesday, August 7, 2018 2:24 AM To: Whaley, Graham <graham.whaley@intel.com> Cc: kata-dev@lists.katacontainers.io Subject: Re: [kata-dev] Intel Gatekeepers for Kata
Good morning. As Eric mentioned in last week's Arch call, the Kata team at Intel are trialing a 'round robin Gatekeeper schedule' for Kata PR review. That is, each week one of our team will prioritise reviewing and moving forwards the open PRs on the Kata repos. This is a process we used for some time with Clear Containers.
We are trying this out to see if we can try and reduce the pending PR list, in
On Mon, Aug 6, 2018 at 4:27 PM, Whaley, Graham <graham.whaley@intel.com> wrote: particular some of the longer standing or more urgent PRs.
This is by no means an Intel-only initiative :-) If you want to be on the rota then
just let me know and I'll add you into the cycle. It would be great if we can try and get at least two nominated reviewers each week, ideally located in different time zones.
Note that to be a gatekeeper you don't have to understand the whole project,
or even have to be the most technical person. Being a gatekeeper is a great opportunity to both get to know the project in more depth and to hone your technical skills. Sometimes all it needs is for somebody to read the list of open PRs and nudge a few along that have been inactive and ask the appropriate domain experts if they could do a review. All volunteers welcome.
If you are new to the project and/or PR reviews, we have a PR review guide
document that is well worth reviewing: https://github.com/kata- containers/community/blob/master/PR-Review-Guide.md .
Today we are sat with 42 open items (you can coax that info out of github via
this link: https://github.com/search?o=asc&q=org%3Akata- containers+is%3Apr+is%3Aopen&s=created&type=Issues).
This week's gatekeeper is .... myself :-)
This is really a good idea. Thanks for bringing it up! Count me in! We really could use some dedicated forces to push forward long running issues/PRs.
And let's not abuse gatekeeper(TM) for it. "review team" and "PR Shepherd" both sound good to me.
Cheers, Tao
-- bergwolf@hyper.sh
Intel Corporation (UK) Limited Registered No. 1134945 (England) Registered Office: Pipers Way, Swindon SN3 1RJ VAT No: 860 2173 47 This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies.