Re: engine-patches mailing list

2015-06-11 Thread Omer Frenkel


- Original Message -
> From: "Itamar Heim" 
> To: "Omer Frenkel" , "infra" 
> Sent: Thursday, June 11, 2015 9:30:10 AM
> Subject: Re: engine-patches mailing list
> 
> On 06/09/2015 01:57 PM, 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?
> 
> 
> they don't have to be subscribed, though we do need to add them as
> approved sender once we see this.
> same for engine-commits, though less than patches on number of folks
> actually making the commit.
> I'm fine with removing engine-patches and keeping engine-commits only
> 

+1 :)
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra


Re: engine-patches mailing list

2015-06-10 Thread Itamar Heim

On 06/09/2015 01:57 PM, 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?



they don't have to be subscribed, though we do need to add them as 
approved sender once we see this.
same for engine-commits, though less than patches on number of folks 
actually making the commit.

I'm fine with removing engine-patches and keeping engine-commits only

___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra


Re: engine-patches mailing list

2015-06-10 Thread Itamar Heim

On 06/09/2015 03:22 PM, David Caro wrote:

On 06/09, Roy Golan wrote:

On 06/09/2015 02:45 PM, Omer Frenkel wrote:


- Original Message -

From: "David Caro" 
To: "Omer Frenkel" 
Cc: "infra" 
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)


its not the same.
registering to the mailing list allows to see everything going through 
in batch.
registering to a project means no difference from patches you are added 
to directly.


having said this - I'm not sure engine-patches is that useful still. 
engine-commits still is.



___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra


Re: engine-patches mailing list

2015-06-09 Thread Roy Golan

On 06/09/2015 03:22 PM, David Caro wrote:

On 06/09, Roy Golan wrote:

On 06/09/2015 02:45 PM, Omer Frenkel wrote:

- Original Message -

From: "David Caro" 
To: "Omer Frenkel" 
Cc: "infra" 
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] https://gerrit.ovirt.org/#/settings/projects




___
Infra mailing list
Infra@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: dc...@redhat.com
Web: www.redhat.com
RHT Global #: 82-62605


___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra


___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra


Re: engine-patches mailing list

2015-06-09 Thread David Caro
On 06/09, Roy Golan wrote:
> On 06/09/2015 02:45 PM, Omer Frenkel wrote:
> >
> >- Original Message -
> >>From: "David Caro" 
> >>To: "Omer Frenkel" 
> >>Cc: "infra" 
> >>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)


> 
> >>>___
> >>>Infra mailing list
> >>>Infra@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: dc...@redhat.com
> >>Web: www.redhat.com
> >>RHT Global #: 82-62605
> >>
> >___
> >Infra mailing list
> >Infra@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: dc...@redhat.com
Web: www.redhat.com
RHT Global #: 82-62605


pgpp8CXl_WSUv.pgp
Description: PGP signature
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra


Re: engine-patches mailing list

2015-06-09 Thread Roy Golan

On 06/09/2015 02:45 PM, Omer Frenkel wrote:


- Original Message -

From: "David Caro" 
To: "Omer Frenkel" 
Cc: "infra" 
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?



___
Infra mailing list
Infra@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: dc...@redhat.com
Web: www.redhat.com
RHT Global #: 82-62605


___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra


___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra


Re: engine-patches mailing list

2015-06-09 Thread David Caro
On 06/09, Omer Frenkel wrote:
> 
> 
> - Original Message -
> > From: "David Caro" 
> > To: "Omer Frenkel" 
> > Cc: "infra" 
> > 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?

This affects the patch review process giving an alternative to subscribing to
that list and easily getting that info from the ui or filtering it out.

> 
> > > ___
> > > Infra mailing list
> > > Infra@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: dc...@redhat.com
> > Web: www.redhat.com
> > RHT Global #: 82-62605
> > 

-- 
David Caro

Red Hat S.L.
Continuous Integration Engineer - EMEA ENG Virtualization R&D

Tel.: +420 532 294 605
Email: dc...@redhat.com
Web: www.redhat.com
RHT Global #: 82-62605


pgpmCdqq5tab0.pgp
Description: PGP signature
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra


Re: engine-patches mailing list

2015-06-09 Thread Omer Frenkel


- Original Message -
> From: "David Caro" 
> To: "Omer Frenkel" 
> Cc: "infra" 
> 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?

> > ___
> > Infra mailing list
> > Infra@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: dc...@redhat.com
> Web: www.redhat.com
> RHT Global #: 82-62605
> 
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra


Re: engine-patches mailing list

2015-06-09 Thread David Caro
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
> ___
> Infra mailing list
> Infra@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: dc...@redhat.com
Web: www.redhat.com
RHT Global #: 82-62605


pgpH5GGaJvHvB.pgp
Description: PGP signature
___
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra