Page MenuHomePhabricator

Undocument running multiple/dedicated pull daemons
ClosedPublic

Authored by epriestley on Jun 26 2014, 1:54 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 6, 7:12 PM
Unknown Object (File)
Thu, Sep 5, 1:12 PM
Unknown Object (File)
Sun, Sep 1, 9:42 AM
Unknown Object (File)
Sun, Sep 1, 9:42 AM
Unknown Object (File)
Sun, Sep 1, 9:42 AM
Unknown Object (File)
Sun, Sep 1, 9:31 AM
Unknown Object (File)
Sun, Aug 25, 4:33 AM
Unknown Object (File)
Mon, Aug 19, 12:05 PM
Subscribers

Details

Summary

In modern Phabricator, we should pretty much handle this automatically, and this advice is no longer very reasonable. See https://github.com/phacility/phabricator/issues/660

Test Plan

Read documentation.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Undocument running multiple/dedicated pull daemons.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, joshuaspence.
joshuaspence edited edge metadata.

Should we also maybe remove the command line options from the daemon?

> ./phd launch RepositoryPullLocal -- A B
> ./phd launch RepositoryPullLocal -- --not A --not B
This revision is now accepted and ready to land.Jun 26 2014, 1:58 AM

I think it's OK to retain support until T4292, which is maaaybe happening in the next couple of months and will require installs with complicated daemon setups to reconfigure things a bit anwyay.

epriestley updated this revision to Diff 23382.

Closed by commit rP3dda0c82f9f4 (authored by @epriestley).