On 06/09, Roy Golan wrote:
> On 06/09/2015 02:45 PM, Omer Frenkel wrote:
>> ----- Original Message -----
>>> From: "David Caro" <dcaroest(a)redhat.com>
>>> To: "Omer Frenkel" <ofrenkel(a)redhat.com>
>>> Cc: "infra" <infra(a)ovirt.org>
>>> Sent: Tuesday, June 9, 2015 2:19:42 PM
>>> Subject: Re: engine-patches mailing list
>>>
>>> On 06/09, Omer Frenkel wrote:
>>>> Hi,
>>>> I've been monitoring the pending requests list for the
engine-patches
>>>> mailing list for a while now,
>>> >from what i could see, no one send mails to this list except automated
>>>> mails initiated at gerrit (which is pretty much expected).
>>>> the rest is spam..
>>>>
>>>> this makes a requirement for people using gerrit (code contributors and
>>>> reviewers) to subscribe to the list,
>>>> and get all the mails on all gerrit activity.
>>>> some people do not want to subscribe, and mails on their activity needs
>>>> manual processing.
>>>>
>>>> maybe there is a way to have automated job to run daily/weekly to add
all
>>>> gerrit users to the accept list for this list?
>>>> then i would change the list to automatically reject all mail from
>>>> non-subscribers and users that are not in the accept list.
>>>>
>>>> what do you think?
>>>> any other suggestion?
>>> see
http://lists.ovirt.org/pipermail/devel/2015-June/010693.html
>> how this affect the engine-patches mailing list administration?
> David why is engine-patches is needed at all? gerrit hooks are the one who
> triggers ci jobs no?
That seems a common misunderstanding, the gerrit hooks are not related to
jenkins, jenkins connects to gerrit itself (has stream access) independently of
the hooks.
Clarified that, not sure why engine-patches is needed but I think itamar and
some team leads use it to monitor activity. Though afaik that can be done
subscribing to the project (maibe that feature was not available on gerrit
before)
True, it wasn't and mails where used for code-review and monitoring.
So all subscribers must know how to subscribe to mail from the gerrit
personal settings [1] and engine-patches should be deprecated
[1]
>>>> _______________________________________________
>>>> Infra mailing list
>>>> Infra(a)ovirt.org
>>>>
http://lists.ovirt.org/mailman/listinfo/infra
>>> --
>>> David Caro
>>>
>>> Red Hat S.L.
>>> Continuous Integration Engineer - EMEA ENG Virtualization R&D
>>>
>>> Tel.: +420 532 294 605
>>> Email: dcaro(a)redhat.com
>>> Web:
www.redhat.com
>>> RHT Global #: 82-62605
>>>
>> _______________________________________________
>> Infra mailing list
>> Infra(a)ovirt.org
>>
http://lists.ovirt.org/mailman/listinfo/infra