Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable per-project email notification setup #67

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mattock
Copy link
Member

@mattock mattock commented Oct 1, 2024

This change allows defining the notification email on a per-project basis.

@mattock
Copy link
Member Author

mattock commented Oct 1, 2024

This was requested by @cron2 some months ago. The current implementation is done with the Scheduler "owner" property:

The Scheduler "owner" property is copied to the Build "owners" property when launching the build. This can be seen from the Build properties pane.

Note that Buidlbot the documentation is wrong - you can't give the owner property a list:

If the "owner cannot be a list of email addresses" is a problem, we can resolve it in a number of ways:

  • Attempt to override the "owners" property of the Build
    • This seemed not to be possible, but is worth another shot
  • Add make local MTA (e.g. postfix) forward emails to multiple addresses
  • Send emails to mailing lists, where applicable
  • Send emails to an email address that can be configured to forward the emails to multiple addresses

This change allows defining the notification email on a per-project basis.

Signed-off-by: Samuli Seppänen <[email protected]>
@mattock
Copy link
Member Author

mattock commented Oct 1, 2024

I sent a trivial documentation fix to the Buildbot project. Hopefully that gets merged soon so that nobody else needs to spend time figuring out what "owner" property needs to look like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants