<br><br><div class="gmail_quote">On Mon, Feb 21, 2011 at 4:25 PM, Todd Willey <span dir="ltr"><<a href="mailto:todd@ansolabs.com">todd@ansolabs.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Good points Ken.  I think we should leave the nova-core as the default<br>
selection.  This lets those go to the nova-core mailing list, and sets<br>
it up so someone/anyone can add reviewers as those requests come in.<br>
<br>
I just tried out the "Claim Review" button on a merge proposal.  It<br>
looks like using it removed the nova-core assignment and replaced it<br>
with me.  I feel I could now add another reviewer (or more) and have<br>
them chime in.  Maybe we could generate a daily report of unclaimed<br>
branches and send it to the list or to someone who had authority to<br>
assign people?<br></blockquote><div><br></div><div>Seems reasonable-ish, I guess I'll just try to express that core developers should being assigning stuff, and when we run into a new-person review we can help them assign appropriate people.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
-todd[1]<br>
<div><div></div><div class="h5"><br>
On Mon, Feb 21, 2011 at 6:52 PM, Ken Pepple <<a href="mailto:ken.pepple@rabbityard.com">ken.pepple@rabbityard.com</a>> wrote:<br>
> On Feb 21, 2011, at 1:34 PM, Andy Smith wrote:<br>
>> Sure, we just need to remove it automatically selecting "Nova Core" and likewise as a reviewer for you so that people are forced to explicitly add a reviewer.<br>
><br>
><br>
> sorry to come in late in the discussion, but i believe this is a bad idea.<br>
><br>
> imho, this raises another barrier between potential developers wanting to contribute and actually contributing. specifically, it assumes that a newbie to the project "knows" a nova-core member.<br>
><br>
> they don't -- i am not sure the nova-core list is even publicly posted.<br>
><br>
> just in my own case, it took me several days of signing contracts, figuring out the toolset/standards, revising the patches, adding myself to wikis and hanging out of IRC before i could get single line of code into nova. at any of those points, i could have simply said *** (expletive deleted) …<br>

><br>
> bottom line, we should be making it easier for people to contribute not harder :)<br>
><br>
> having said that, i understand what you are trying to do.<br>
> so, as a counter proposal, can we implement a round-robin or smart routing (based on file names or modules) reviewer assignment ?<br>
> failing that, could we assign everything to ttx (sorry) and he can assign reviewers ?<br>
> failing that, could we give them a list of potential reviewers (with some good instructions) to choose from ?<br>
><br>
> please correct me if i misconstrued your proposal<br>
> /k<br>
</div></div><div><div></div><div class="h5">> _______________________________________________<br>
> Mailing list: <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
> Post to     : <a href="mailto:openstack@lists.launchpad.net">openstack@lists.launchpad.net</a><br>
> Unsubscribe : <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
> More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
><br>
</div></div></blockquote></div><br>