openstack team mailing list archive
-
openstack team
-
Mailing list archive
-
Message #08593
Re: Gerrit minimum review time frame
Obviously, a better review algorithm is required,
To avoid hungry condition of some branches.
Now the whole system feels more like a client/server model.
Maybe a peer to peer like model behaves better.
在 2012-3-13,15:13,"Jesse Andrews" <anotherjesse@xxxxxxxxx> 写道:
> Maybe it is just me, but most reviews seem to take hours to days to
> complete. I'm been sitting here waiting for a one line change to get
> a second "+2 / approved" so I can redeploy our test cluster for the
> last 2 hours.
>
> When lots of people are active, reviews can be approved in quickly -
> especially when people reach out to others who are familiar with the
> code in question.
>
> Do we need a "time-gate" or can you use a feed-reader and
> https://github.com/openstack/nova/commits/master.atom to read patches
> were approved while one is away/sleeping/time off? (opening bugs or
> asking questions in IRC is always acceptable regardless of how long
> the review took)
>
> Delaying by 2 hours on a weekday at noon vs 2am saturday night?
> Would we then start debating how best to gate at different times based
> on when people are available?
>
> That said if the complexity added is minimal I don't think it would
> hurt - it is another thing that our (small) CI team has to do.
>
> Jesse
>
> On Mon, Mar 12, 2012 at 11:51 PM, Mark McLoughlin <markmc@xxxxxxxxxx> wrote:
>> On Mon, 2012-03-12 at 17:57 -0500, Josh Kearney wrote:
>>> Is this is really a problem that needs solving? I'd like to believe that no
>>> member of Nova Core would approve something that they aren't familiar with.
>>
>> I think the issue is more a case of allowing some time for other
>> reviewers to come along and catch issues before it gets merged.
>>
>> Sometimes there can be a few folks working through an issue together and
>> the patch gets pushed and approved so quickly that no-one else gets a
>> chance to review.
>>
>>> IMHO, we should be making better attempts at not letting branches sit
>>> around for days/weeks at a time.
>>>
>>> On Mon, Mar 12, 2012 at 4:59 PM, Joe Gordon <jogo@xxxxxxxxxxxxxxxx> wrote:
>>>
>>>> Hi All,
>>>>
>>>> I have noticed that some Gerrit branches get approved very quickly,
>>>> sometimes in a matter of minutes. While most of the time these branches
>>>> are vetted properly, the window for reviewing can be so small that a
>>>> non-trivial branch lands but without enough vetting. If someone is in a
>>>> meeting for half on hour they may miss the entire review window. To fix
>>>> this problem I propose a minimum time frame (should be overridable in an
>>>> emergency) for a branch to be approved, perhaps 2 hours. This time frame
>>>> would start on 'Upload time.'
>>
>> I like the idea, personally. Even in the case of "emergencies", I don't
>> think 2 hours is unreasonable.
>>
>> Cheers,
>> Mark.
>>
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~openstack
>> Post to : openstack@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~openstack
>> More help : https://help.launchpad.net/ListHelp
>
> _______________________________________________
> Mailing list: https://launchpad.net/~openstack
> Post to : openstack@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~openstack
> More help : https://help.launchpad.net/ListHelp