Page MenuHomePhabricator

[drydock/core] Allow nested leasing and resource allocation
AbandonedPublic

Authored by hach-que on Jul 1 2015, 4:21 AM.
Referenced Files
Unknown Object (File)
Thu, Apr 25, 3:20 AM
Unknown Object (File)
Wed, Apr 24, 7:43 AM
Unknown Object (File)
Fri, Apr 5, 6:46 PM
Unknown Object (File)
Thu, Apr 4, 3:06 PM
Unknown Object (File)
Wed, Apr 3, 7:12 PM
Unknown Object (File)
Mon, Apr 1, 1:53 AM
Unknown Object (File)
Dec 26 2023, 6:12 PM
Unknown Object (File)
Dec 22 2023, 8:48 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T2015: Implement Drydock
Summary

Ref T2015. This allows for Drydock blueprints to lease and acquire resources during their own leasing or resource acquisition. This is reqiured for working copies, which must lease a host resource.

Test Plan

Tested as part of the working copy changes.

Diff Detail

Repository
rP Phabricator
Branch
hachque-reconstructed
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7781
Build 8594: [Placeholder Plan] Wait for 30 Seconds
Build 8593: arc lint + arc unit

Event Timeline

hach-que retitled this revision from to [drydock/core] Allow nested leasing and resource allocation.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
hach-que edited edge metadata.

Rebase on origin/master

epriestley edited edge metadata.

I think this is obsoleted by the general de-nesting of allocation and acquisition and the clarification of locking/constraint responsibility.

Specifically, it is now expected that the allocator may build a resource for lease A and have it stolen by lease B before it can acquire a lease on it (which is fine, it will just retry a little later). And it is no longer expected that a single process can push an allocation through by performing less work than a generic taskmaster -- that is, we don't know which tasks need to be completed in order to complete an allocation or acquisition, only that doing work from the queue will eventually result in success if there are no actual configuration/implementation problems.

HEAD also now has a WorkingCopy blueprint which appears to work, lease host resources correctly, and not have race/constraint issues.

Let me know if I missed something and this is still relevant?

This revision now requires changes to proceed.Sep 23 2015, 6:00 PM