Page MenuHomePhabricator

Send emails to subscribers when builds pass or fail
AbandonedPublic

Authored by hach-que on Mar 24 2014, 4:31 AM.
Tags
None
Referenced Files
F14116700: D8609.diff
Thu, Nov 28, 4:28 PM
F14115388: D8609.id20399.diff
Thu, Nov 28, 11:31 AM
Unknown Object (File)
Wed, Nov 6, 7:31 PM
Unknown Object (File)
Oct 19 2024, 2:08 AM
Unknown Object (File)
Oct 10 2024, 5:50 AM
Unknown Object (File)
Oct 9 2024, 3:45 PM
Unknown Object (File)
Oct 9 2024, 3:45 PM
Unknown Object (File)
Oct 7 2024, 10:57 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T1049: Implement Harbormaster
Summary

Ref T1049. This is a very basic implementation of sending emails to subscribers of build plans when the build passes or fails. At the moment it doesn't embed any information other than a link to a build (e.g. it doesn't pull in the container, buildable or plan name).

It is probably also missing logic to check whether the subscriber can see the buildable / container that the email is about. Since the only thing that's included right now is a link, it shouldn't be too harmful, but a policy check definitely needs to be added when the logic changes to pull in build information (such as plan and buildable).

Test Plan

Added a mailer implementation that writes the mail information to a Phabricator file and verified that the message arrives correctly (I can't send an actual email out from my development machine).

Diff Detail

Repository
rP Phabricator
Branch
harbormaster-emails
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

hach-que retitled this revision from to Send emails to subscribers when builds pass or fail.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.

The new application transactions for Harbormaster negate the need for this diff.