Page MenuHomePhabricator

Fix command typo in managing_daemons.diviner
ClosedPublic

Authored by jevripio on Jun 4 2014, 8:26 PM.
Tags
None
Referenced Files
F18111547: D9374.id.diff
Tue, Aug 12, 11:08 AM
F18082298: D9374.id22354.diff
Tue, Aug 5, 5:09 AM
F17918944: D9374.id.diff
Jul 30 2025, 1:50 AM
F17917869: D9374.id22344.diff
Jul 30 2025, 12:17 AM
F17870420: D9374.diff
Jul 28 2025, 1:54 PM
Unknown Object (File)
Jul 3 2025, 1:08 AM
Unknown Object (File)
Jun 20 2025, 2:14 PM
Unknown Object (File)
Jun 9 2025, 2:21 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).