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
F19929169: D19073.diff
Wed, Apr 8, 2:17 PM
F19928914: D19073.diff
Wed, Apr 8, 1:06 PM
F19822084: D19073.diff
Mar 7 2026, 2:41 AM
F19383804: D19073.id45729.diff
Dec 27 2025, 1:22 AM
F19266353: D19073.id.diff
Dec 22 2025, 7:59 PM
F19242576: D19073.diff
Dec 21 2025, 11:06 PM
F18827321: D19073.id.diff
Oct 24 2025, 9:21 AM
F18793051: D19073.id.diff
Oct 16 2025, 8:39 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