Differential D15702

Move Aphlict logging and PID configuration options to config file
ClosedPublic
Actions

Authored by epriestley on Apr 13 2016, 6:46 PM.

Tags

None

Referenced Files

	F15489428: D15702.id.diff
	Fri, Apr 11, 9:00 AM

	F15485870: D15702.diff
	Thu, Apr 10, 3:12 AM

	F15469872: D15702.diff
	Fri, Apr 4, 5:35 PM

	F15417533: D15702.id37845.diff
	Thu, Mar 20, 5:27 PM

	F15417532: D15702.id37829.diff
	Thu, Mar 20, 5:27 PM

	F15417531: D15702.id.diff
	Thu, Mar 20, 5:27 PM

	F15381944: D15702.id37829.diff
	Fri, Mar 14, 10:01 AM

	F15312105: D15702.id37829.diff
	Mar 6 2025, 2:00 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T10697: Move Aphlict server configuration to a separate configuration file

Commits

rPc6b092595497: Move Aphlict logging and PID configuration options to config file

Summary

Ref T10697. Mostly straightforward. Also allow the server to have multiple logs and log options in the future (e.g., different verbosities or separate admin/client logs or whatever). No specific plans for this, but the default log is pretty noisy today.

Test Plan

Set up a couple of logs, started server, saw it log to them.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 37829.Apr 13 2016, 6:46 PM

epriestley retitled this revision from to Move Aphlict logging and PID configuration options to config file.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: chad.

epriestley added a task: T10697: Move Aphlict server configuration to a separate configuration file.

🙈🙈🙈

This revision is now accepted and ready to land.Apr 13 2016, 6:51 PM

Closed by commit rPc6b092595497: Move Aphlict logging and PID configuration options to config file (authored by epriestley, committed by epriestley). · Explain WhyApr 14 2016, 11:55 AM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in T10751: Make Phabricator Highly Available.Apr 14 2016, 5:50 PM

Revision Contents
Changeset List

Path

Size

conf/

aphlict/

aphlict.default.json

8 lines

src/

applications/

aphlict/

management/

PhabricatorAphlictManagementWorkflow.php

62 lines

config/

check/

PhabricatorExtraConfigSetupCheck.php

2 lines

option/

PhabricatorNotificationConfigOptions.php

7 lines

docs/

user/

configuration/

notifications.diviner

11 lines

support/

aphlict/

server/

aphlict_server.js

20 lines

Diff 37845

conf/aphlict/aphlict.default.json

Loading...

src/applications/aphlict/management/PhabricatorAphlictManagementWorkflow.php

Loading...

src/applications/config/check/PhabricatorExtraConfigSetupCheck.php

Loading...

src/applications/config/option/PhabricatorNotificationConfigOptions.php

Loading...

src/docs/user/configuration/notifications.diviner

Loading...

support/aphlict/server/aphlict_server.js

Loading...