Differential D9374

Fix command typo in managing_daemons.diviner
ClosedPublic
Actions

Authored by jevripio on Jun 4 2014, 8:26 PM.

Tags

None

Referenced Files

	F14379185: D9374.diff
	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

	Unknown Object (File)
	Nov 4 2024, 10:07 PM

Subscribers

Details

Reviewers

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

Branch

fix/managing_daemons_example

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 847
Build 847: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

jevripio updated this revision to Diff 22344.Jun 4 2014, 8:26 PM

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)

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 4 2014, 8:26 PM

Herald added a subscriber: epriestley. · View Herald Transcript

Harbormaster completed remote builds in B847: Diff 22344.Jun 4 2014, 8:27 PM

Thanks!

This revision is now accepted and ready to land.Jun 4 2014, 10:40 PM

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

Revision Contents
Changeset List

Path

Size

src/

docs/

user/

configuration/

managing_daemons.diviner

2 lines

Diff 22344

src/docs/user/configuration/managing_daemons.diviner

Loading...