Page MenuHomePhabricator

Strip some obsolete code out of Drydock
ClosedPublic

Authored by epriestley on Sep 23 2015, 7:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 12, 5:33 PM
Unknown Object (File)
Thu, Apr 11, 8:53 AM
Unknown Object (File)
Wed, Apr 10, 7:51 PM
Unknown Object (File)
Mon, Apr 8, 10:05 PM
Unknown Object (File)
Mon, Apr 1, 4:09 PM
Unknown Object (File)
Sun, Mar 31, 12:55 PM
Unknown Object (File)
Mar 4 2024, 6:15 PM
Unknown Object (File)
Mar 4 2024, 6:15 PM
Subscribers
None

Details

Summary

Ref T9252. This simplifies some Drydock code.

Most of this code relates to the old notion of Drydock being able to enumerate all the tasks it needs to complete in order to acquire a lease. The code has stepped back from this, since it's unnecessary, the queue is more powerful than it used to be, and it would be a lot of work to keep track of.

The ~only thing that should ever wait for leases in modern code is bin/drydock lease, and it's fine for it to just sit there sleeping, so this just does that.

This reduces the granularity of logging, but I'll address that separately in future logging-focused changes.

Test Plan

Used bin/drydock lease to acquire a lease, saw it acquire cleanly.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Strip some obsolete code out of Drydock.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, hach-que.
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 23 2015, 7:45 PM
This revision was automatically updated to reflect the committed changes.