Page MenuHomePhabricator

Fix command typo in managing_daemons.diviner
ClosedPublic

Authored by jevripio on Jun 4 2014, 8:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 21, 6:12 AM
Unknown Object (File)
Sat, Dec 21, 2:41 AM
Unknown Object (File)
Sun, Dec 8, 6:50 PM
Unknown Object (File)
Sun, Dec 8, 1:12 AM
Unknown Object (File)
Fri, Dec 6, 1:36 PM
Unknown Object (File)
Wed, Dec 4, 8:07 PM
Unknown Object (File)
Nov 20 2024, 7:34 PM
Unknown Object (File)
Nov 11 2024, 5:20 AM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP52fb3eb5f403: Fix command typo in managing_daemons.diviner
Summary

Fix the 'managing daemons' doc's example for launching daemons in a
way suitable for multiple web servers. Separate the '--no-discovery'
argument with '--', otherwise it doesn't appear to be understood.

Test Plan

Attempt to launch daemon in various ways, make sure to include
ways that are expected to fail, otherwise it may not be doing what
we expect.

phd launch PhabricatorRepositoryPullLocalDaemon --no-discovery
.. errors in daemon log 'unrecognised argument' ..
phd launch PhabricatorRepositoryPullLocalDaemon -- --not-an-arg
.. errors in daemon log 'unrecognised argument' ..
phd launch PhabricatorRepositoryPullLocalDaemon -- --no-discovery
.. no errors in daemon log ..

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

jevripio retitled this revision from to Fix command typo in managing_daemons.diviner.
jevripio updated this object.
jevripio edited the test plan for this revision. (Show Details)
This revision is now accepted and ready to land.Jun 4 2014, 10:40 PM
epriestley updated this revision to Diff 22354.

Closed by commit rP52fb3eb5f403 (authored by @jevripio, committed by @epriestley).