Differential D4190

Accept then discard mails from Phabricator instances
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F53868983: D4190.id10991.diff
	Mon, Oct 12, 21:35

	F53868948: D4190.id10992.diff
	Mon, Oct 12, 21:35

	F53778367: D4190.id10991.diff
	Mon, Oct 12, 05:56

	F53755308: D4190.id10993.diff
	Mon, Oct 12, 01:48

	Unknown Object (File)
	Sat, Oct 10, 13:16

	Unknown Object (File)
	Fri, Oct 9, 23:46

	Unknown Object (File)
	Thu, Oct 8, 11:13

	Unknown Object (File)
	Wed, Oct 7, 09:29

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...