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
F13233247: D19613.diff
Tue, May 21, 2:08 AM
Unknown Object (File)
Thu, May 2, 3:24 AM
Unknown Object (File)
Thu, Apr 25, 1:29 AM
Unknown Object (File)
Apr 11 2024, 8:59 AM
Unknown Object (File)
Apr 5 2024, 6:01 PM
Unknown Object (File)
Mar 10 2024, 12:13 PM
Unknown Object (File)
Feb 3 2024, 7:19 PM
Unknown Object (File)
Dec 1 2023, 9:05 AM
View All 58 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
Branch
drydock1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20692
Build 28124: Run Core Tests
Build 28123: arc lint + arc unit

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 46887

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