Differential D19043

Don't run older mail setup checks if "cluster.mailers" is configured
ClosedPublic
Actions

Authored by epriestley on Feb 9 2018, 1:50 AM.

Tags

None

Referenced Files

	F18390597: D19043.diff
	Fri, Aug 29, 7:00 AM

	F18336211: D19043.id45659.diff
	Mon, Aug 25, 10:14 AM

	F17939820: D19043.id45658.diff
	Jul 31 2025, 3:28 AM

	F17923713: D19043.id.diff
	Jul 30 2025, 8:37 AM

	F17899158: D19043.diff
	Jul 29 2025, 8:32 AM

	F17868683: D19043.diff
	Jul 28 2025, 8:25 AM

	F17596314: D19043.id45658.diff
	Jul 8 2025, 10:38 AM

	Unknown Object (File)
	Jun 29 2025, 7:52 AM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T12677: Support multiple mail delivery services for automatic failover
Commits: rP09b446b269f5: Don't run older mail setup checks if "cluster.mailers" is configured

Summary

Ref T12677. Skip these checks if we're doing the new stuff. Also, allow priority to be unspecified.

Test Plan

Will deploy.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 9 2018, 1:50 AM

This revision was not accepted when it landed; it landed in state Draft.Feb 9 2018, 1:51 AM

Closed by commit rP09b446b269f5: Don't run older mail setup checks if "cluster.mailers" is configured (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

Harbormaster failed remote builds in B19453: Diff 45658!Feb 9 2018, 1:53 AM

Harbormaster completed remote builds in B19453: Diff 45658.Feb 9 2018, 2:02 AM

Revision Contents
Changeset List

Path

Size

src/

applications/

config/

check/

PhabricatorMailSetupCheck.php

4 lines

infrastructure/

cluster/

config/

PhabricatorClusterMailersConfigType.php

4 lines

Diff 45659

View Options

src/applications/config/check/PhabricatorMailSetupCheck.php