Page MenuHomePhabricator
Differential D9610

Garbage collect daemon logs
ClosedPublic
Actions

Authored by epriestley on Jun 17 2014, 10:27 PM.
Tags
None
Referenced Files
F19820493: D9610.id23037.diff
Fri, Mar 6, 1:57 PM
F19673591: D9610.id23034.diff
Sun, Feb 8, 6:14 AM
F19534082: D9610.id23037.diff
Jan 20 2026, 10:25 PM
F19517051: D9610.diff
Jan 15 2026, 7:04 AM
F19208711: D9610.diff
Dec 16 2025, 11:23 PM
F18848307: D9610.id.diff
Oct 30 2025, 4:19 AM
F18847064: D9610.diff
Oct 29 2025, 10:44 PM
F18785807: D9610.id23034.diff
Oct 14 2025, 11:57 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