[Engine-devel] [vdsm] stale gerrit patches

[Itamar Heim]

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)?

>
> Maintainers can close patches that are no interest nor progress.
>
> Alon
>