Page MenuHomePhabricator
Differential D9610

Garbage collect daemon logs
ClosedPublic
Actions

Authored by epriestley on Jun 17 2014, 10:27 PM.
Tags
None
Referenced Files
F13996388: D9610.id.diff
Wed, Oct 23, 8:48 PM
F13993790: D9610.diff
Wed, Oct 23, 1:28 AM
F13991804: D9610.id23037.diff
Tue, Oct 22, 12:53 PM
F13982348: D9610.id23034.diff
Sat, Oct 19, 10:33 PM
Unknown Object (File)
Sep 21 2024, 3:48 AM
Unknown Object (File)
Aug 28 2024, 5:50 PM
Unknown Object (File)
Aug 16 2024, 11:47 PM
Unknown Object (File)
Aug 15 2024, 8:01 AM
View All Files
Subscribers
epriestley

Details

Reviewers
joshuaspence
Commits
Restricted Diffusion Commit
rPdf3736e81e56: Garbage collect daemon logs
Summary

We already have GC for daemon log events, but not for daemon logs themselves.

Collect old daemon logs which aren't still running.

Test Plan

Ran phd debug garbage, observed old logs get cleaned up. Started some daemons, re-ran garbage, made sure they stuck around.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Diff 23034.Jun 17 2014, 10:27 PM
epriestley retitled this revision from to Garbage collect daemon logs.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: joshuaspence.
Herald added a subscriber: epriestley. · View Herald TranscriptJun 17 2014, 10:27 PM
Harbormaster completed remote builds in B1195: Diff 23034.Jun 17 2014, 10:28 PM
joshuaspence accepted this revision.Jun 17 2014, 10:29 PM
joshuaspence edited edge metadata.
This revision is now accepted and ready to land.Jun 17 2014, 10:29 PM
epriestley closed this revision.Jun 17 2014, 10:33 PM
epriestley updated this revision to Diff 23037.

Closed by commit rPdf3736e81e56 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
2 lines
src/
2 lines
applications/
daemon/
garbagecollector/
PhabricatorDaemonLogEventGarbageCollector.php
PhabricatorDaemonLogGarbageCollector.php
2 lines
7 lines

Diff 23037

View Options

resources/sql/autopatches/20140617.daemonlog.sql

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/daemon/garbagecollector/PhabricatorDaemonLogEventGarbageCollector.php

Loading...
View Options

src/applications/daemon/garbagecollector/PhabricatorDaemonLogGarbageCollector.php

Loading...
Phabricator · Built by Phacility