One extra concern. Since I work in the UK, most things happen while I am sleeping.
Not sure I know of a good solution to that problem. A 12 hour window seems stupidly long, and I don't fancy becoming nocturnal. Having said that, the core reviewers look to be doing a fine job at the moment, so I am not too worried. John > -----Original Message----- > From: [email protected] > [mailto:[email protected]] > On Behalf Of Andrew Hutchings > Sent: 13 March 2012 09:37 > To: [email protected] > Subject: Re: [Openstack] Gerrit minimum review time frame > > Hi Joe, > > On 12/03/12 21:59, Joe Gordon wrote: > > 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 can maybe see this working with the larger projects (although I would > imagine for us in CI it would be a lot of work implementing this for little > gain). > But with smaller teams you can find that sometimes the coding is done with > all the core team involved. In this case code review for small changes will > only take minutes at most. > > Kind Regards > -- > Andrew Hutchings - LinuxJedi - http://www.linuxjedi.co.uk/ > > _______________________________________________ > Mailing list: https://launchpad.net/~openstack > Post to : [email protected] > Unsubscribe : https://launchpad.net/~openstack > More help : https://help.launchpad.net/ListHelp _______________________________________________ Mailing list: https://launchpad.net/~openstack Post to : [email protected] Unsubscribe : https://launchpad.net/~openstack More help : https://help.launchpad.net/ListHelp
