Page MenuHomePhabricator

Measure how long build targets take in Harbormaster
ClosedPublic

Authored by hach-que on Aug 7 2014, 7:28 AM.
Tags
None
Referenced Files
F14007064: D10174.diff
Mon, Oct 28, 11:00 PM
F14002144: D10174.id.diff
Fri, Oct 25, 3:41 PM
F14000910: D10174.diff
Fri, Oct 25, 2:19 AM
Unknown Object (File)
Thu, Oct 10, 12:45 AM
Unknown Object (File)
Tue, Oct 8, 11:50 PM
Unknown Object (File)
Tue, Oct 8, 11:50 PM
Unknown Object (File)
Tue, Oct 8, 11:50 PM
Unknown Object (File)
Mon, Oct 7, 6:46 PM

Details

Summary

Ref T1049. This keeps track of how long a build target takes to execute in Harbormaster and displays it in the build view page. I'm not sure whether "Started" is really that useful once the target has completed?

Also, I change the name of the time taken depending on whether or not the target has completed; if it's still in progress it's called "Elapsed" and if it's completed then it's "Duration". The primary reason for this is that "Duration" sounds like post tense, whereas "Elapsed" is current tense. I'm not sure whether this is okay or not?

Test Plan

Ran a Sleep build step and saw the target dates / times appear correctly.

Diff Detail

Repository
rP Phabricator
Branch
time-targets
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2079
Build 2081: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

hach-que retitled this revision from to Measure how long build targets take in Harbormaster.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/harbormaster/controller/HarbormasterBuildViewController.php
115

(Not specific to this diff and we aren't very consistent about it, but prefer PhabricatorTime::getNow() to time(), since we can change the value of "now" in unit tests with the PhabricatorTime API. We might also synchronize "time" to the database (or some other source of truth) in the future to try to deal with clock skew.)

This revision is now accepted and ready to land.Aug 11 2014, 9:39 PM

(And all the UI stuff seems fine to me, we can tweak it when the UI gets some attention.)

hach-que updated this revision to Diff 24604.

Closed by commit rPefc82c727bb4 (authored by @hach-que).