Reducing email notifications to infra ML

Description

At the moment, there are email notifications sent to infra ML which are not required. Those can be dropped from the yaml.

For example (according to :
Build-artifacts jobs shouldn't email infra@ovirt.org.

Activity

Show:

Eyal Edri December 14, 2018 at 3:09 PM

was the patch merged? can we close this?

Eyal Edri November 29, 2018 at 9:01 AM

for build artifacts jobs, I don't see a reason why infra should be notifed and not just the maintainer or his team. same as check-patch emails.

Emil Natan November 29, 2018 at 8:56 AM

It's not as simple as removing infra@ovirt.org from all aliases. Maybe there are people relying on receiving these and there are aliases which include only infra so you can't simple remove it. So maybe infra could be replaced with some other alias or single email addresses, but one needs to send email to infra with all details and propose plan to restructure these aliases.

Eyal Edri November 28, 2018 at 3:31 PM

git clone jenkins repo
cd jobs/confs/projects

git grep infra@ovirt.org
defaults.yaml: email-to: 'infra@ovirt.org'
ovirt-appliance/ovirt-appliance.yaml: email-to: 'infra@ovirt.org, yturgema@redhat.com'
ovirt-node-ng/ovirt-node-ng.yaml: email-to: 'infra@ovirt.org, yturgema@redhat.com'
ovirt-node-ng/ovirt-node-ng.yaml: email-to: 'infra@ovirt.org, yturgema@redhat.com'
ovirt/change-queues.yaml: email-recipients: 'infra@ovirt.org'
ovirt/system-tests.yaml: email-to: infra@ovirt.org
system/system_backup.yaml: mail_to: infra@ovirt.org
system/system_jobs.yaml: recipients: 'infra@ovirt.org'

Liora Milbaum November 28, 2018 at 3:28 PM

I get them as part of the infra ML

Barak Korren November 28, 2018 at 2:23 PM

Can you be specific? Which emails?

Build-artifacts jobs for example do not send emails anywhere...

Fixed

Details

Assignee

Reporter

Priority

Created November 28, 2018 at 12:07 PM
Updated August 29, 2019 at 2:12 PM
Resolved December 16, 2018 at 8:50 AM