Page MenuHomePhabricator

Show lease on Repository Operation detail view and awaken on failures
ClosedPublic

Authored by epriestley on Oct 26 2015, 5:21 PM.
Tags
None
Referenced Files
F13324918: D14341.diff
Sat, Jun 15, 1:07 AM
F13311605: D14341.diff
Tue, Jun 11, 5:16 AM
F13311433: D14341.id34626.diff
Tue, Jun 11, 4:53 AM
F13298403: D14341.diff
Fri, Jun 7, 6:44 AM
F13261718: D14341.diff
Mon, May 27, 1:22 AM
F13256367: D14341.id34626.diff
Sat, May 25, 11:26 AM
F13247393: D14341.id34626.diff
Thu, May 23, 6:53 PM
F13247392: D14341.id34625.diff
Thu, May 23, 6:53 PM
Subscribers
None

Details

Summary

Ref T182. Couple of minor improvements here:

  • Show the Drydock lease when viewing a Repository Operation detail screen. This just makes it easier to jump around between relevant objects.
  • When tasks are waiting for a lease, awaken them when it breaks or is released, not just when it is acquired. This makes the queue move forward faster when errors occur.
Test Plan
  • Viewed a repository operation and saw a link to the lease.
  • Did a bad land (intentional merge problem) and got an error in about ~3 seconds instead of ~17.

Diff Detail

Repository
rP Phabricator
Branch
workmerge3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8419
Build 9677: Run Core Tests
Build 9676: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Show lease on Repository Operation detail view and awaken on failures.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Oct 26 2015, 7:36 PM
epriestley edited edge metadata.

Clean update.

This revision was automatically updated to reflect the committed changes.