Differential D4190

Accept then discard mails from Phabricator instances
ClosedPublic
Actions

Authored by dereckson on Aug 23 2026, 19:25.

Tags

None

Referenced Files

	F52629759: D4190.id10993.diff
	Sat, Oct 3, 16:52

	Unknown Object (File)
	Fri, Oct 2, 20:47

	Unknown Object (File)
	Thu, Oct 1, 11:01

	Unknown Object (File)
	Mon, Sep 28, 15:28

	Unknown Object (File)
	Mon, Sep 28, 12:40

	Unknown Object (File)
	Sun, Sep 27, 16:22

	Unknown Object (File)
	Fri, Sep 25, 09:03

	Unknown Object (File)
	Thu, Sep 24, 08:15

View All 92 Files

Subscribers

None

Details

Reviewers

Maniphest Tasks

T2349: Create mail blackhole / silently discard mail to notifications-noreply@devcentral.nasqueron.org

Commits

rOPS8d066385e6b2: Accept then discard mails from Phabricator instances

Summary

Test Plan

check logs for Phabricator no-reply mails
ensure regular virtual mailboxes are still working

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Passed

Unit

No Test Coverage

Branch

postfix-blackhole

Build Status

Buildable 6839
Build 7127: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Aug 23 2026, 19:25

dereckson created this revision.

Harbormaster completed remote builds in B6838: Diff 10991.Aug 23 2026, 19:25

Coherent variables names

Harbormaster completed remote builds in B6839: Diff 10992.Aug 23 2026, 19:27

dereckson accepted this revision.Aug 23 2026, 20:43

This revision is now accepted and ready to land.Aug 23 2026, 20:43

dereckson mentioned this in T2349: Create mail blackhole / silently discard mail to notifications-noreply@devcentral.nasqueron.org.Aug 23 2026, 20:45

Closed by commit rOPS8d066385e6b2: Accept then discard mails from Phabricator instances (authored by dereckson). · Explain WhyAug 23 2026, 20:48

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rOPS8d066385e6b2: Accept then discard mails from Phabricator instances.

dereckson added inline comments.Aug 23 2026, 21:14

roles/mailserver/postfix/config.sls

Version deployed has a uncommitted fix on Complector for this typo.

diff --git a/roles/mailserver/postfix/config.sls b/roles/mailserver/postfix/config.sls
index df8796fa..95b3f0e0 100644
--- a/roles/mailserver/postfix/config.sls
+++ b/roles/mailserver/postfix/config.sls
@@ -26,7 +26,7 @@
     - source: salt://roles/mailserver/postfix/files/blackhole.cf
     - template: jinja
     - context:
-        discard_mailboxes: {{ pillar["postfix_config"].get("discard_mailboxes", [])
+        discard_mailboxes: {{ pillar["postfix_config"].get("discard_mailboxes", []) }}
 
 {{ dirs.etc }}/postfix/postfix-to-mailman.py:
   file.managed:

Revision Contents
Changeset List

Path

Size

pillar/

mailserver/

5 lines

roles/

mailserver/

postfix/

7 lines

files/

19 lines

6 lines

Diff 10992

pillar/mailserver/postfix.sls

Loading...

roles/mailserver/postfix/config.sls

Loading...

roles/mailserver/postfix/files/blackhole.cf

Loading...

roles/mailserver/postfix/files/main.cf

Loading...