Paths

Table of Contentst

Differential D10385

Tweak --limit for bin/phd log
ClosedPublic
Actions

Authored by epriestley on Aug 29 2014, 12:31 PM.

Details

Reviewers

btrahan

Maniphest Tasks

T5405: Move daemon logs from the web UI to console-only

Commits

Restricted Diffusion Commit
rP340b40172fb9: Tweak --limit for bin/phd log

Summary

Ref T5405.

--limit wasn't actually used anywhere.
Make it mean "the N newest lines".

Test Plan

Ran bin/phd log, bin/phd log --limit 3.

Diff Detail

Repository

rP Phabricator

Branch

loglimit

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2389
Build 2393: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 24993.Aug 29 2014, 12:31 PM

epriestley retitled this revision from to Tweak --limit for bin/phd log.

epriestley updated this object.

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

epriestley added a reviewer: btrahan.

epriestley added a task: T5405: Move daemon logs from the web UI to console-only.

Herald added a subscriber: epriestley. · View Herald TranscriptAug 29 2014, 12:31 PM

Harbormaster completed remote builds in B2389: Diff 24993.Aug 29 2014, 12:31 PM

btrahan accepted this revision.Aug 29 2014, 5:00 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Aug 29 2014, 5:00 PM

Closed by commit rP340b40172fb9 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

daemon/

management/

PhabricatorDaemonManagementLogWorkflow.php

14 lines

Diff 24993

View Options

src/applications/daemon/management/PhabricatorDaemonManagementLogWorkflow.php

Tweak --limit for bin/phd logClosedPublicActions