<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Mon, Apr 22, 2013 at 10:13 AM, Russell Bryant <span dir="ltr"><<a href="mailto:rbryant@redhat.com" target="_blank">rbryant@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 04/22/2013 12:22 PM, Dolph Mathews wrote:<br>
> After spending some time on other projects, termie is back and<br>
> contributing to keystone once more. For those that don't know him, he<br>
> rearchitected & rewrote keystone's implementation during essex into what<br>
> it is today. I'd be happy to see him rejoin the list of core<br>
> contributors for Havana.<br>
><br>
> Reviews: <a href="https://review.openstack.org/#/q/reviewer:termie,n,z" target="_blank">https://review.openstack.org/#/q/reviewer:termie,n,z</a><br>
> Contributions: <a href="https://review.openstack.org/#/q/owner:termie,n,z" target="_blank">https://review.openstack.org/#/q/owner:termie,n,z</a><br>
><br>
> He's also working on an OAuth 1.0a extension for Havana.<br>
><br>
> Comments, +1's and termie-esque -2's appreciated within the week.<br>
<br>
+1 fwiw, but this triggered another set of thoughts I wanted to bring up ...<br>
<br>
I'd like to see us all try to speak about the core teams as review teams<br>
as opposed to recognition of being an important contributor.</blockquote><div><br></div><div style>+10; I wouldn't recommend someone for core if I didn't trust their ability to review code with a knowledgable eye towards maintaining stability, backwards compatibility, test coverage, user experience, dev-friendliness, etc.</div>
<div style><br></div><div style>Having a history of great contributions is merely a bonus that reflects their knowledge of the code base.</div><div style><br></div><div style>Ultimately, "core contributor" to me simply means that this person's downvotes on code reviews are consistently well thought out and meaningful, such that an upvote by the same person shows a lot of confidence in the patch.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think<br>
many people (and their managers) see membership on whatever-core as the<br>
ultimate goal, and lack of membership on whatever-core meaning they are<br>
not doing well enough.<br>
<br>
In the case of nova, our review team (nova-core) is up to 20 members.<br>
That's quite large and I'd like to avoid making it any bigger any time<br>
soon.  That means that there are quite a few people that do amazing work<br>
contributing to the project, but are not in the subset that is<br>
responsible for approving code.  I don't want it to seem like their<br>
contributions are not valued.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Russell Bryant<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</font></span></blockquote></div><br></div></div>