Page MenuHomePhabricator

When `bin/drydock lease` is interrupted, release leases
ClosedPublic

Authored by epriestley on Feb 13 2018, 12:24 PM.
Tags
None
Referenced Files
F15464591: D19073.id45717.diff
Wed, Apr 2, 11:07 AM
F15464189: D19073.id45729.diff
Wed, Apr 2, 7:08 AM
F15437932: D19073.id45729.diff
Tue, Mar 25, 10:12 PM
F15437738: D19073.id.diff
Tue, Mar 25, 9:13 PM
F15433253: D19073.id.diff
Mon, Mar 24, 10:17 PM
F15429238: D19073.id.diff
Mon, Mar 24, 1:17 AM
F15429097: D19073.diff
Mon, Mar 24, 12:34 AM
F15420639: D19073.diff
Fri, Mar 21, 3:42 PM
Subscribers
Restricted Owners Package

Details

Summary

Depends on D19072. Ref T13073. Currently, you can leave leases stranded by using ^C to interrupt the script. Handle signals and release leases on destruction if they haven't activated yet.

Also, print out more useful information before and after activation.

Test Plan

Mashed ^C while runnning bin/drydock lease ... --trace, saw the lease release.

Diff Detail

Repository
rP Phabricator
Branch
dry4
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/drydock/management/DrydockManagementLeaseWorkflow.php:92XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 19520
Build 26414: Run Core Tests
Build 26413: arc lint + arc unit