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
F19082511: D8609.id.diff
Tue, Dec 2, 11:24 AM
F19074015: D8609.diff
Mon, Dec 1, 3:55 AM
F18856837: D8609.id.diff
Nov 1 2025, 11:45 AM
F18851786: D8609.diff
Oct 31 2025, 4:08 AM
F18780754: D8609.id.diff
Oct 12 2025, 7:33 PM
F18724726: D8609.diff
Sep 30 2025, 3:58 AM
F18673404: D8609.diff
Sep 25 2025, 10:29 AM
F18627766: D8609.id20399.diff
Sep 16 2025, 4:44 AM
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.