4:52 a.m.
----- Original Message -----
From: "Zhou Zheng Sheng"
<zhshzhou(a)linux.vnet.ibm.com>
To: "Ayal Baron" <abaron(a)redhat.com>
Cc: "engine-devel" <engine-devel(a)ovirt.org>,
vdsm-devel(a)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(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.
>
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(a)linux.vnet.ibm.com
Telephone: 86-10-82454397
_______________________________________________
Engine-devel mailing list
Engine-devel(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-devel