Page MenuHomePhabricator
Differential D10377

Clean up lease directories when lease is released
AbandonedPublic
Actions

Authored by epriestley on Aug 28 2014, 1:24 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 21, 7:16 PM
Unknown Object (File)
Fri, Dec 13, 1:19 PM
Unknown Object (File)
Mon, Dec 9, 10:56 AM
Unknown Object (File)
Sun, Dec 8, 10:43 PM
Unknown Object (File)
Nov 23 2024, 4:01 AM
Unknown Object (File)
Nov 19 2024, 4:13 PM
Unknown Object (File)
Nov 14 2024, 6:07 AM
Unknown Object (File)
Nov 14 2024, 4:37 AM
View All Files
Subscribers
champo
epriestley
joshuaspence
Korvin

Details

Reviewers
hach-que
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T6015: Harbormaster build not cleaning lease directory after build finishes
T2015: Implement Drydock
Summary

Depends on D10304. This cleans up leased directories on the preallocated blueprint after the lease is released.

Test Plan

Not tested yet.

Diff Detail

Repository
rP Phabricator
Branch
delete-leased-directories
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2415
Build 9136: [Placeholder Plan] Wait for 30 Seconds
Build 2419: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

hach-que updated this revision to Diff 24976.Aug 28 2014, 1:24 AM
hach-que retitled this revision from to Clean up lease directories when lease is released.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a parent revision: D10304: [drydock/core] Implement resource constraints and correct locking mechanisms with yielding for Drydock.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptAug 28 2014, 1:24 AM
Herald added a subscriber: epriestley. · View Herald Transcript
Harbormaster completed remote builds in B2380: Diff 24976.Aug 28 2014, 1:24 AM
hach-que added a task: T2015: Implement Drydock.Aug 30 2014, 8:44 AM
hach-que added a task: T6015: Harbormaster build not cleaning lease directory after build finishes.Sep 2 2014, 5:31 AM
epriestley accepted this revision.Sep 2 2014, 6:06 PM
epriestley added a reviewer: epriestley.
This revision is now accepted and ready to land.Sep 2 2014, 6:06 PM
hach-que updated this revision to Diff 25033.Sep 3 2014, 7:45 AM
hach-que edited edge metadata.

Prevent crashes when files are locked during cleanup

Herald added a subscriber: Korvin. · View Herald TranscriptSep 3 2014, 7:45 AM
Harbormaster completed remote builds in B2415: Diff 25033.Sep 3 2014, 7:46 AM
champo added a subscriber: champo.Jan 26 2015, 5:35 PM
joshuaspence added a subscriber: joshuaspence.Feb 27 2015, 6:02 AM
epriestley commandeered this revision.Sep 25 2015, 3:31 AM
epriestley edited reviewers, added: hach-que; removed: epriestley.

I believe this is obsolete at HEAD. Leasing a host no longer implies directory creation and WorkingCopy resources do resets / cleanup.

This revision now requires review to proceed.Sep 25 2015, 3:31 AM
epriestley abandoned this revision.Sep 25 2015, 3:31 AM

Revision Contents
Changeset List

PathSize
src/
applications/
drydock/
blueprint/
15 lines

Diff 25033

View Options

src/applications/drydock/blueprint/DrydockPreallocatedHostBlueprintImplementation.php

Loading...
Phabricator · Built by Phacility