[Engine-devel] [vdsm] stale gerrit patches

Yaniv Bronheim ybronhei at redhat.com
Sun Sep 29 09:52:53 UTC 2013



----- Original Message -----
> From: "Zhou Zheng Sheng" <zhshzhou at linux.vnet.ibm.com>
> To: "Ayal Baron" <abaron at redhat.com>
> Cc: "engine-devel" <engine-devel at ovirt.org>, vdsm-devel at lists.fedorahosted.org
> Sent: Wednesday, September 25, 2013 5:16:16 AM
> Subject: Re: [Engine-devel] [vdsm] stale gerrit patches
> 
> 
> 
> on 2013/09/24 05:21, Ayal Baron wrote:
> > 
> > 
> > ----- Original Message -----
> >>
> >>
> >> ----- 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: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 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
> >>>
> >>> 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.
> > 
> 
> At least we all agree on that old patches should be abandoned.
> 
> I think we can do this in a semi-automatic way. A cron job checks the
> patch's freshness, and sends an email to warn the author and reviewers
> of an old patch. If the someone has a good reason to keep the patch, he
> can leave a comment on the gerrit web page saying "I want to #keep the
> patch# because ...". Then the system skips the patches whose last
> comment contains "#keep the patch#". If no one cares it, the patch is
> abandoned after some time.

+1 for Zhou Zheng Sheng.
Much better suggestion than automatically forgetting old patches by removing them.
A reminder can be sent after couple of weeks or even a month, and auto abandon the patch if no response added to the bug within a week.

I like this suggestion if we want to add automation for this process (as we all prefer automation when possible), and it'll probably help a bit to clean our gerrit dash board

> --
> Thanks and best regards!
> 
> Zhou Zheng Sheng / 周征晟
> E-mail: zhshzhou at linux.vnet.ibm.com
> Telephone: 86-10-82454397
> 
> _______________________________________________
> Engine-devel mailing list
> Engine-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel
>



More information about the Devel mailing list