[Engine-devel] [vdsm] stale gerrit patches

[Alon Bar-Lev]

----- Original Message -----
> From: "Alon Bar-Lev" <alonbl at redhat.com>
> To: "Itamar Heim" <iheim at redhat.com>
> Cc: "David Caro" <dcaroest at redhat.com>, "engine-devel" <engine-devel at ovirt.org>, vdsm-devel at lists.fedorahosted.org
> Sent: Monday, September 23, 2013 1:52:56 PM
> Subject: Re: [vdsm] stale gerrit patches
> 
> 
> 
> ----- Original Message -----
> > From: "Itamar Heim" <iheim at redhat.com>
> > To: "Alon Bar-Lev" <alonbl at redhat.com>
> > Cc: "David Caro" <dcaroest at redhat.com>, "engine-devel"
> > <engine-devel at ovirt.org>, vdsm-devel at lists.fedorahosted.org
> > 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" <iheim at redhat.com>
> > >> To: "David Caro" <dcaroest at redhat.com>
> > >> Cc: "engine-devel" <engine-devel at ovirt.org>,
> > >> vdsm-devel at lists.fedorahosted.org
> > >> 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
> > >>>> vdsm-devel at lists.fedorahosted.org
> > >>>> 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
> > >
> > 
> > 
>