----- Original Message ----- > From: "Itamar Heim" <[email protected]> > To: "Alon Bar-Lev" <[email protected]> > Cc: "David Caro" <[email protected]>, "engine-devel" > <[email protected]>, [email protected] > Sent: Monday, September 23, 2013 1:50:35 PM > Subject: Re: [vdsm] stale gerrit patches > > On 09/23/2013 01:49 PM, Alon Bar-Lev wrote: > > > > > > ----- Original Message ----- > >> From: "Itamar Heim" <[email protected]> > >> To: "David Caro" <[email protected]> > >> Cc: "engine-devel" <[email protected]>, > >> [email protected] > >> Sent: Monday, September 23, 2013 1:47:47 PM > >> Subject: Re: [vdsm] stale gerrit patches > >> > >> On 09/23/2013 01:46 PM, David Caro wrote: > >>> On Mon 23 Sep 2013 12:36:58 PM CEST, Itamar Heim wrote: > >>>> we have some very old gerrit patches. > >>>> I'm for abandoning patches which were not touched over 60 days (to > >>>> begin with, I think the number should actually be lower). > >>>> they can always be re-opened by any interested party post their closure. > >>>> > >>>> i.e., looking at gerrit, the patch list should actually get attention, > >>>> and not be a few worth looking at, with a "lot of old patches" > >>>> > >>>> thoughts? > >>>> > >>>> Thanks, > >>>> Itamar > >>>> _______________________________________________ > >>>> vdsm-devel mailing list > >>>> [email protected] > >>>> https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel > >>> > >>> It might helpful to have a cron-like script that checks the age of the > >>> posts and first notifies the sender, the reviewers and the maintainer, > >>> and if the patch is not updated in a certain period just abandons it. > >>> > >> > >> yep - warn after X days via email to just owner (or all subscribed to > >> the patch), and close if no activity for X+14 days or something like that. > > > > This will be annoying. > > > > And there are patches that pending with good reason. > > pending for 60 days with zero activity on them (no comment, no rebase, > nothing)?
http://gerrit.ovirt.org/#/q/status:open+project:ovirt-engine+branch:master+topic:independent_deployments,n,z > > > > > Maintainers can close patches that are no interest nor progress. > > > > Alon > > > > _______________________________________________ vdsm-devel mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
