Paths

Table of Contentst

Differential D373

Aphlict configuration upgrade
ClosedPublic
Actions

Authored by dereckson on May 20 2016, 18:51.

Tags

Referenced Files

	F3777673: D373.id889.diff
	Mon, Nov 25, 02:10

	F3777671: D373.id890.diff
	Mon, Nov 25, 02:10

	F3777635: D373.id892.diff
	Mon, Nov 25, 02:00

	F3777557: D373.id891.diff
	Mon, Nov 25, 01:47

	F3777476: D373.id893.diff
	Mon, Nov 25, 01:34

	F3776907: D373.diff
	Sun, Nov 24, 23:44

	Unknown Object (File)
	Tue, Nov 19, 23:07

	Unknown Object (File)
	Tue, Nov 19, 19:10

Subscribers

Nasqueron Docker deployment squad

Details

Reviewers

Maniphest Tasks

T829: Upgrade Aphlict image to new Phabricator configuration

Commits

rDAPHLICTbe98e0f29a95: Aphlict configuration upgrade

Summary

Phabricator changed the way Aphlict must be configured.

Background information could be found
at https://secure.phabricator.com/T10794.

Fixes T829.

Test Plan

Relaunch an aphlict container for devcentral.nasqueron.org
and phabricator.wolfplex.be.

Diff Detail

Repository

rDAPHLICT Docker image for Aphlict

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson updated this revision to Diff 889.May 20 2016, 18:51

dereckson added a task: T829: Upgrade Aphlict image to new Phabricator configuration.

dereckson retitled this revision from to Aphlict configuration upgrade.

dereckson updated this object.

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

dereckson added a reviewer: Sandlayth.

Herald added a project: Docker images. · View Herald TranscriptMay 20 2016, 18:51

Herald added a subscriber: Nasqueron Docker deployment squad. · View Herald Transcript

the server doesn't run as root by default
/var/run/aphlict.pid and /var/log/aphlict.log must be writable by app

So launcher line is node --max-old-space-size=256 -- /opt/phabricator/support/aphlict/server/aphlict_server.js --config=/opt/phabricator/conf/aphlict/aphlict.custom.json

User app created (reusing nasqueron/nginx-php-fpm uid/gid)
Node command adjusted

README has to be updated: we've logs \o/

README.md
44–47	It does.

Test works:

README updated

Fixed indentation issue.

Sandlayth accepted this revision.May 20 2016, 20:21

Sandlayth edited edge metadata.

This revision is now accepted and ready to land.May 20 2016, 20:21

Closed by commit rDAPHLICTbe98e0f29a95: Aphlict configuration upgrade (authored by dereckson, committed by dereckson). · Explain WhyMay 20 2016, 20:35

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

14 lines

50 lines

aphlict.custom.json

24 lines

Diff 893

Dockerfile

Loading...

README.md

Loading...

aphlict.custom.json

Loading...