Page MenuHomePhabricator

Stop build target start times from being overwritten on reentry
ClosedPublic

Authored by epriestley on Apr 27 2018, 6:26 PM.
Tags
None
Referenced Files
F15507649: D19412.id.diff
Tue, Apr 15, 9:53 PM
F15503621: D19412.id46433.diff
Mon, Apr 14, 11:27 AM
F15458983: D19412.id46426.diff
Mon, Mar 31, 8:55 AM
F15450002: D19412.id.diff
Fri, Mar 28, 1:25 PM
F15431827: D19412.diff
Mon, Mar 24, 2:44 PM
F15399475: D19412.id46433.diff
Mar 17 2025, 4:43 AM
F15380981: D19412.diff
Mar 14 2025, 5:18 AM
F15380466: D19412.id46426.diff
Mar 14 2025, 3:10 AM
Subscribers
Restricted Owners Package

Details

Summary

See PHI615. Ref T13130. An install is reporting that "Lease Working Copy" build steps always report "Built instantly" after completion.

I'm not 100% sure that this is the fix, but I'm like 99% sure: "Lease Working Copy" build steps yield after they ask Drydock for a lease. They will later reenter doWork(), see that the lease is filled, and complete.

Right now, we reset the start time every time we enter doWork(). Instead, set it only if it hasn't been set yet.

Test Plan

This is low-risk and a bit tricky to reproduce locally, but I'll run some production builds and see what they look like.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable