Page MenuHomePhabricator
Differential D19613

Add an "Activated Epoch" and an "Acquired Epoch" to Drydock Leases
ClosedPublic
Actions

Authored by epriestley on Aug 27 2018, 6:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Feb 9, 6:23 AM
Unknown Object (File)
Sun, Feb 9, 6:23 AM
Unknown Object (File)
Sun, Feb 9, 6:23 AM
Unknown Object (File)
Tue, Jan 28, 4:07 AM
Unknown Object (File)
Mon, Jan 27, 5:43 PM
Unknown Object (File)
Sun, Jan 26, 9:56 PM
Unknown Object (File)
Sat, Jan 25, 2:04 AM
Unknown Object (File)
Sat, Jan 25, 2:04 AM
View All Files
Subscribers
Restricted Owners Package

Details

Reviewers
amckinley
Maniphest Tasks
T13189: Plans: 2018 Week 35 Bonus Content
Commits
rP2f5c6541fc8e: Add an "Activated Epoch" and an "Acquired Epoch" to Drydock Leases
Summary

Ref T13189. See PHI690. When a lease is first acquired or activated, note the time. This supports better visibility into queue lengths. For now, this is only queryable via DB and visible in the UI, but can be more broadly exposed in the future.

Test Plan

Landed a revision, saw the leases get sensible timestamps for acquisition/activation.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Aug 27 2018, 6:23 PM
Owners added a subscriber: Restricted Owners Package.Aug 27 2018, 6:23 PM
Harbormaster completed remote builds in B20692: Diff 46887.Aug 27 2018, 6:24 PM
epriestley requested review of this revision.Aug 27 2018, 6:24 PM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 27 2018, 6:27 PM
Closed by commit rP2f5c6541fc8e: Add an "Activated Epoch" and an "Acquired Epoch" to Drydock Leases (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in T13189: Plans: 2018 Week 35 Bonus Content.Aug 27 2018, 6:32 PM

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
2 lines
2 lines
src/
applications/
drydock/
controller/
24 lines
storage/
20 lines

Diff 46888

View Options

resources/sql/autopatches/20180827.drydock.01.acquired.sql

Loading...
View Options

resources/sql/autopatches/20180827.drydock.02.activated.sql

Loading...
View Options

src/applications/drydock/controller/DrydockLeaseViewController.php

Loading...
View Options

src/applications/drydock/storage/DrydockLease.php

Loading...
Phabricator · Built by Phacility