Page MenuHomePhabricator
Differential D9610

Garbage collect daemon logs
ClosedPublic
Actions

Authored by epriestley on Jun 17 2014, 10:27 PM.
Tags
None
Referenced Files
F18848307: D9610.id.diff
Thu, Oct 30, 4:19 AM
F18847064: D9610.diff
Wed, Oct 29, 10:44 PM
F18785807: D9610.id23034.diff
Oct 14 2025, 11:57 AM
F18778106: D9610.diff
Oct 11 2025, 8:05 AM
F18711495: D9610.id.diff
Sep 29 2025, 4:14 AM
F18708860: D9610.diff
Sep 28 2025, 11:00 PM
F18383024: D9610.diff
Aug 28 2025, 10:29 PM
F17918017: D9610.id23034.diff
Jul 30 2025, 12:29 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