openstack team mailing list archive
-
openstack team
-
Mailing list archive
-
Message #00881
Re: Review days for nova-core members
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?
-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
>
Follow ups
References
-
Review days for nova-core members
From: Soren Hansen, 2011-02-16
-
Re: Review days for nova-core members
From: Paul Voccio, 2011-02-17
-
Re: Review days for nova-core members
From: Soren Hansen, 2011-02-17
-
Re: Review days for nova-core members
From: Thierry Carrez, 2011-02-17
-
Re: Review days for nova-core members
From: Jay Pipes, 2011-02-17
-
Re: Review days for nova-core members
From: Soren Hansen, 2011-02-17
-
Re: Review days for nova-core members
From: Andy Smith, 2011-02-18
-
Re: Review days for nova-core members
From: Thierry Carrez, 2011-02-21
-
Re: Review days for nova-core members
From: Andy Smith, 2011-02-21
-
Re: Review days for nova-core members
From: Ken Pepple, 2011-02-21