----- Original Message ----- > From: "Alon Bar-Lev" <[email protected]> > To: "Itamar Heim" <[email protected]> > Cc: "David Caro" <[email protected]>, "engine-devel" > <[email protected]>, [email protected] > Sent: Monday, September 23, 2013 1:52:56 PM > Subject: Re: [vdsm] stale gerrit patches > > > > ----- 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
http://gerrit.ovirt.org/#/q/status:open+project:ovirt-engine+branch:master+topic:ldap_independence,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
