← Back to team overview

openstack team mailing list archive

Re: Review days for nova-core members

 

On Mon, Feb 21, 2011 at 4:25 PM, Todd Willey <todd@xxxxxxxxxxxx> wrote:

> Good points Ken.  I think we should leave the nova-core as the default
> selection.  This lets those go to the nova-core mailing list, and sets
> it up so someone/anyone can add reviewers as those requests come in.
>
> I just tried out the "Claim Review" button on a merge proposal.  It
> looks like using it removed the nova-core assignment and replaced it
> with me.  I feel I could now add another reviewer (or more) and have
> them chime in.  Maybe we could generate a daily report of unclaimed
> branches and send it to the list or to someone who had authority to
> assign people?
>

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.


>
> -todd[1]
>
> On Mon, Feb 21, 2011 at 6:52 PM, Ken Pepple <ken.pepple@xxxxxxxxxxxxxx>
> wrote:
> > On Feb 21, 2011, at 1:34 PM, Andy Smith wrote:
> >> 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.
> >
> >
> > sorry to come in late in the discussion, but i believe this is a bad
> idea.
> >
> > 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.
> >
> > they don't -- i am not sure the nova-core list is even publicly posted.
> >
> > 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) …
> >
> > bottom line, we should be making it easier for people to contribute not
> harder :)
> >
> > having said that, i understand what you are trying to do.
> > so, as a counter proposal, can we implement a round-robin or smart
> routing (based on file names or modules) reviewer assignment ?
> > failing that, could we assign everything to ttx (sorry) and he can assign
> reviewers ?
> > failing that, could we give them a list of potential reviewers (with some
> good instructions) to choose from ?
> >
> > please correct me if i misconstrued your proposal
> > /k
> > _______________________________________________
> > Mailing list: https://launchpad.net/~openstack
> > Post to     : openstack@xxxxxxxxxxxxxxxxxxx
> > Unsubscribe : https://launchpad.net/~openstack
> > More help   : https://help.launchpad.net/ListHelp
> >
>

References