1:17 a.m.
----- Original Message -----
From: "Ayal Baron" <abaron(a)redhat.com>
To: "Alon Bar-Lev" <alonbl(a)redhat.com>
Cc: "Itamar Heim" <iheim(a)redhat.com>, "engine-devel"
<engine-devel(a)ovirt.org>, vdsm-devel(a)lists.fedorahosted.org
Sent: Tuesday, September 24, 2013 12:21:23 AM
Subject: Re: [vdsm] stale gerrit patches
----- Original Message -----
>
>
> ----- Original Message -----
> > From: "Itamar Heim" <iheim(a)redhat.com>
> > To: "Alon Bar-Lev" <alonbl(a)redhat.com>
> > Cc: "David Caro" <dcaroest(a)redhat.com>,
"engine-devel"
> > <engine-devel(a)ovirt.org>, vdsm-devel(a)lists.fedorahosted.org
> > Sent: Monday, September 23, 2013 1:54:39 PM
> > Subject: Re: [vdsm] stale gerrit patches
> >
> > On 09/23/2013 01:52 PM, Alon Bar-Lev wrote:
> > >
> > >
> > > ----- Original Message -----
> > >> From: "Itamar Heim" <iheim(a)redhat.com>
> > >> To: "Alon Bar-Lev" <alonbl(a)redhat.com>
> > >> Cc: "David Caro" <dcaroest(a)redhat.com>,
"engine-devel"
> > >> <engine-devel(a)ovirt.org>, vdsm-devel(a)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(a)redhat.com>
> > >>>> To: "David Caro" <dcaroest(a)redhat.com>
> > >>>> Cc: "engine-devel" <engine-devel(a)ovirt.org>,
> > >>>> vdsm-devel(a)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(a)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:maste...
> >
> > so how does it help us to have these patches, some without any comment
> > from any reviewer.
> > lets get them reviewed and decide one way or the other, rather than let
> > them get old and stay forever
>
> Again... maintainer can close these if he likes.
> Owner can close these if he likes.
right, but why?
a patch without activity being abandoned might actually spur someone into
motion (rebasing and resubmitting, prodding maintainers etc).
I'm +1 for automatically abandoning old patches.
I do not understand why maintainer should not have human interaction with its
contributers.
>
> The problem is that maintainers avoid closing.
> And that there are people who submitted patches without CC anyone and gone.
>
> So a simple logic can be applied after we add metadata into tree:
>
> 1. If no maintainer is CCed on change, close that change within short cycle
> (can be even a week).
> 2. Maintainer to close patches that have no interest in.
>
> >
> > >
> > >>
> > >>>
> > >>> Maintainers can close patches that are no interest nor progress.
> > >>>
> > >>> Alon
> > >>>
> > >>
> > >>
> >
> >
> _______________________________________________
> vdsm-devel mailing list
> vdsm-devel(a)lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
>