Page MenuHomePhabricator

Improve error and exception handling for Drydock leases
ClosedPublic

Authored by epriestley on Oct 1 2015, 12:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 18, 3:56 AM
Unknown Object (File)
Mon, Dec 16, 10:38 AM
Unknown Object (File)
Tue, Dec 10, 6:00 PM
Unknown Object (File)
Sun, Dec 8, 10:29 PM
Unknown Object (File)
Thu, Dec 5, 9:01 AM
Unknown Object (File)
Wed, Dec 4, 1:10 PM
Unknown Object (File)
Mon, Dec 2, 11:48 AM
Unknown Object (File)
Mon, Dec 2, 11:48 AM
Subscribers
None

Details

Summary

Ref T9252. See companion change in D14211. This does the same thing for leases.

Particularly, most of the TODOs about error handling can just be removed because they'll do the right things by default now.

This and D14211 also move slot lock release to after resource destruction. This feels cleaner than trying to release early at release/break.

Test Plan

Restarted a Harbormaster build, got a clean build result. This needs more vetting but I'll clean up any issues as I hit them.

Diff Detail

Repository
rP Phabricator
Branch
dd13
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8121
Build 9266: [Placeholder Plan] Wait for 30 Seconds
Build 9265: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Improve error and exception handling for Drydock leases.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, hach-que.
hach-que edited edge metadata.
This revision is now accepted and ready to land.Oct 1 2015, 12:59 PM
This revision was automatically updated to reflect the committed changes.