<p dir="ltr">Thanks Brendan. Given the projects usage of Gerrit for most of repos, would there be an advantage to using Gerrit to enforce the reviewer and maintainer roles? How will we prevent MAINTAINERS from getting out of sync with commit privileges?</p>
<br><div class="gmail_quote"><div dir="ltr">On Tue, Nov 1, 2016, 1:04 AM Brendan Higgins <<a href="mailto:brendanhiggins@google.com">brendanhiggins@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">For those who were not present, there was an OpenBMC meetup of sorts<br class="gmail_msg">
last week. One issue that Nancy Yuen, Abhishek Pandit, and myself<br class="gmail_msg">
brought up for non-IBM contributors was ownership and organization of<br class="gmail_msg">
projects; this went in a couple directions, but at some point somebody<br class="gmail_msg">
proposed a Linux kernel style MAINTAINERS list as a way to find people<br class="gmail_msg">
relevant for discussions in the various sub-parts of the project.<br class="gmail_msg">
<br class="gmail_msg">
I made an initial implementation: <a href="https://gerrit.openbmc-project.xyz/#/c/944/" rel="noreferrer" class="gmail_msg" target="_blank">https://gerrit.openbmc-project.xyz/#/c/944/</a><br class="gmail_msg">
<br class="gmail_msg">
Most of it should be pretty self-explanatory (if not please join in<br class="gmail_msg">
the discussion). Nevertheless, I distinguished between reviewers and<br class="gmail_msg">
maintainers; whereas a reviewer is someone who is familiar enough with<br class="gmail_msg">
a given part of a project that she is a good person to address<br class="gmail_msg">
questions to concerning that component and should be included on code<br class="gmail_msg">
reviews; a maintainer is someone who's approval is required to check a<br class="gmail_msg">
patch set into a repository.<br class="gmail_msg">
<br class="gmail_msg">
I chose reviewers based on the amount of commits contributed to a<br class="gmail_msg">
given repository; I know this is not the best way to decide whether<br class="gmail_msg">
someone should be considered an expert on a particular part of the<br class="gmail_msg">
code base, but I thought it was a reasonable place to start.<br class="gmail_msg">
Additionally, breaking things up only at the repo level is probably<br class="gmail_msg">
also not the most useful as people may become experts within a<br class="gmail_msg">
particular part of a given repository, but again I think this is a<br class="gmail_msg">
reasonable starting place and I added semantics for breaking it down<br class="gmail_msg">
further.<br class="gmail_msg">
<br class="gmail_msg">
For maintainers, I chose two people for each repository; the first is<br class="gmail_msg">
the person who already has the authoritative access I described above;<br class="gmail_msg">
the second was chosen to be a major contributor. For Linux, u-boot,<br class="gmail_msg">
ipmi-tool, and qemu that is some combination of Joel, Cédric, and<br class="gmail_msg">
Jeremy. For everything else, all of the userland stuff and yocto, that<br class="gmail_msg">
is Patrick and Brad. I recall discussing adding Brad as a maintainer<br class="gmail_msg">
for the userland stuff in the meeting, but I figure it is something we<br class="gmail_msg">
should also discuss on the email list for everyone's benefit. We had<br class="gmail_msg">
not discussed adding a second maintainer for anything else, but I<br class="gmail_msg">
figure it is probably best practice to have more than once person who<br class="gmail_msg">
can submit code in all cases.<br class="gmail_msg">
<br class="gmail_msg">
I fully expect that the MAINTAINERS file I uploaded will require a<br class="gmail_msg">
good deal of discussion before it is ready to submit, but I figured it<br class="gmail_msg">
would be easiest to start with something to talk over.<br class="gmail_msg">
<br class="gmail_msg">
Cheers<br class="gmail_msg">
_______________________________________________<br class="gmail_msg">
openbmc mailing list<br class="gmail_msg">
<a href="mailto:openbmc@lists.ozlabs.org" class="gmail_msg" target="_blank">openbmc@lists.ozlabs.org</a><br class="gmail_msg">
<a href="https://lists.ozlabs.org/listinfo/openbmc" rel="noreferrer" class="gmail_msg" target="_blank">https://lists.ozlabs.org/listinfo/openbmc</a><br class="gmail_msg">
</blockquote></div>