Paths

Table of Contentst

Differential D9208

Show daemon arguments with `./bin/phd status`.
ClosedPublic
Actions

Authored by joshuaspence on May 19 2014, 11:00 PM.

Tags

None

Referenced Files

	F13711671: D9208.id21872.diff
	Wed, Aug 28, 10:27 PM

	F13710944: D9208.id21870.diff
	Wed, Aug 28, 6:58 PM

	F13710417: D9208.id.diff
	Wed, Aug 28, 5:31 PM

	F13708898: D9208.diff
	Wed, Aug 28, 10:01 AM

	Unknown Object (File)
	Mon, Aug 26, 6:22 PM

	Unknown Object (File)
	Sun, Aug 25, 2:27 PM

	Unknown Object (File)
	Sun, Aug 25, 12:18 PM

	Unknown Object (File)
	Wed, Aug 21, 5:52 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T4735: `phd status` should show daemon arguments

Commits

Restricted Diffusion Commit
rPeba11238b921: Show daemon arguments with `./bin/phd status`.

Summary

Fixes T4735. When running ./bin/phd, show daemon arguments.

Test Plan

./bin/phd status
PID  	Started                 	Daemon                                            Arguments
12711	May 20 2014, 9:02:52 AM 	PhabricatorRepositoryPullLocalDaemon              []
12716	May 20 2014, 9:02:52 AM 	PhabricatorGarbageCollectorDaemon                 []
12733	May 20 2014, 9:02:53 AM 	PhabricatorTaskmasterDaemon                       []
12768	May 20 2014, 9:02:53 AM 	PhabricatorTaskmasterDaemon                       []
12775	May 20 2014, 9:02:53 AM 	PhabricatorTaskmasterDaemon                       []
12780	May 20 2014, 9:02:54 AM 	PhabricatorTaskmasterDaemon                       []
12838	May 20 2014, 9:02:54 AM 	PhabricatorFactDaemon                             []
13436	May 20 2014, 9:03:23 AM 	PhabricatorRepositoryPullLocalDaemon              ["X","--not","Y"]

Diff Detail

Repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

joshuaspence updated this revision to Diff 21870.May 19 2014, 11:00 PM

joshuaspence retitled this revision from to Show daemon arguments with `./bin/phd status`..

joshuaspence updated this object.

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

joshuaspence added a reviewer: epriestley.

joshuaspence added a task: T4735: `phd status` should show daemon arguments.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 19 2014, 11:00 PM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

Harbormaster completed remote builds in B585: Diff 21870.May 19 2014, 11:01 PM

json_encode the arguments

Harbormaster completed remote builds in B586: Diff 21871.May 19 2014, 11:02 PM

Reduce the width of the "Daemon" column.

joshuaspence edited the test plan for this revision. (Show Details)May 19 2014, 11:05 PM

Harbormaster completed remote builds in B587: Diff 21872.May 19 2014, 11:05 PM

joshuaspence added a parent revision: D9207: Store daemon arguments in reference file..May 20 2014, 10:27 PM

As with D9205, maybe csprintf + readable argument? I'm onboard with this if that sounds like a pain.

joshuaspence added a parent revision: D9229: Add `%LR` support to `csprintf`..May 20 2014, 11:34 PM

Use csprintf('%LR', $daemon->getArgv()) instead of json_encode($daemon->getArgv()).

Harbormaster completed remote builds in B616: Diff 21919.May 20 2014, 11:36 PM

epriestley accepted this revision.May 20 2014, 11:47 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.May 20 2014, 11:47 PM

Closed by commit rPeba11238b921 (authored by @joshuaspence, committed by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

daemon/

management/

PhabricatorDaemonManagementStatusWorkflow.php

14 lines

infrastructure/

daemon/

control/

PhabricatorDaemonReference.php

6 lines

Diff 21921

src/applications/daemon/management/PhabricatorDaemonManagementStatusWorkflow.php

Loading...

src/infrastructure/daemon/control/PhabricatorDaemonReference.php

Loading...