Page MenuHomePhabricator
Differential D10371

Daemons - move combined log to console
ClosedPublic
Actions

Authored by btrahan on Aug 27 2014, 9:42 PM.
Tags
None
Referenced Files
F18818909: D10371.id.diff
Wed, Oct 22, 3:11 AM
F18746776: D10371.id.diff
Fri, Oct 3, 1:25 PM
F18742009: D10371.id24963.diff
Thu, Oct 2, 4:53 PM
F18712145: D10371.diff
Mon, Sep 29, 5:17 AM
F18673618: D10371.id24961.diff
Thu, Sep 25, 11:22 AM
F18647838: D10371.diff
Sep 19 2025, 2:56 PM
F18646641: D10371.diff
Sep 19 2025, 10:34 AM
F18631023: D10371.id.diff
Sep 16 2025, 1:57 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T5405: Move daemon logs from the web UI to console-only
Commits
Restricted Diffusion Commit
rP06882a99cf92: Daemons - move combined log to console
Summary

Fixes T5405.

Test Plan

ran a few commands (log, log --id X --id Y, log --id BADX, log --id BADX --id BADY) and verified good output

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan updated this revision to Diff 24961.Aug 27 2014, 9:42 PM
btrahan added a task: T5405: Move daemon logs from the web UI to console-only.
btrahan retitled this revision from to Daemons - move combined log to console.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
Herald added subscribers: Korvin, epriestley. · View Herald TranscriptAug 27 2014, 9:42 PM
btrahan edited the test plan for this revision. (Show Details)Aug 27 2014, 9:42 PM
Harbormaster completed remote builds in B2371: Diff 24961.Aug 27 2014, 9:42 PM
epriestley accepted this revision.Aug 27 2014, 9:45 PM
epriestley edited edge metadata.

Cool, looks reasonable to me.

This revision is now accepted and ready to land.Aug 27 2014, 9:45 PM
btrahan closed this revision.Aug 27 2014, 9:50 PM
btrahan updated this revision to Diff 24963.

Closed by commit rP06882a99cf92 (authored by @btrahan).

Revision Contents
Changeset List

PathSize
src/
2 lines
applications/
daemon/
application/
1 line
controller/
1 line
management/
62 lines
infrastructure/
internationalization/
translation/
4 lines

Diff 24963

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/daemon/application/PhabricatorDaemonsApplication.php

Loading...
View Options

src/applications/daemon/controller/PhabricatorDaemonCombinedLogController.php

Loading...
View Options

src/applications/daemon/controller/PhabricatorDaemonController.php

Loading...
View Options

src/applications/daemon/management/PhabricatorDaemonManagementLogWorkflow.php

Loading...
View Options

src/infrastructure/internationalization/translation/PhabricatorBaseEnglishTranslation.php

Loading...
Phabricator · Built by Phacility