Differential D14341

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

Authored by epriestley on Oct 26 2015, 5:21 PM.

Tags

None

Referenced Files

	F14031413: D14341.diff
	Sat, Nov 9, 10:21 AM

	F14026386: D14341.id34626.diff
	Fri, Nov 8, 1:15 AM

	F14026385: D14341.id34625.diff
	Fri, Nov 8, 1:15 AM

	F14026384: D14341.id34610.diff
	Fri, Nov 8, 1:15 AM

	F14026383: D14341.id.diff
	Fri, Nov 8, 1:15 AM

	F14026382: D14341.diff
	Fri, Nov 8, 1:15 AM

	F14015631: D14341.id34625.diff
	Sun, Nov 3, 10:53 PM

	F14000214: D14341.diff
	Thu, Oct 24, 8:09 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T182: Commit into repository directly from differential

Commits

Restricted Diffusion Commit
rP2326d5f8d090: Show lease on Repository Operation detail view and awaken on failures

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 34610.Oct 26 2015, 5:21 PM

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.

epriestley added a task: T182: Commit into repository directly from differential.

chad accepted this revision.Oct 26 2015, 7:36 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Oct 26 2015, 7:36 PM

Clean update.

Closed by commit rP2326d5f8d090: Show lease on Repository Operation detail view and awaken on failures (authored by epriestley, committed by epriestley). · Explain WhyOct 26 2015, 8:00 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

drydock/

controller/

DrydockRepositoryOperationViewController.php

9 lines

storage/

DrydockLease.php

18 lines

DrydockRepositoryOperation.php

9 lines

worker/

DrydockLeaseUpdateWorker.php

18 lines

DrydockRepositoryOperationUpdateWorker.php

2 lines

Diff 34626

src/applications/drydock/controller/DrydockRepositoryOperationViewController.php

Loading...

src/applications/drydock/storage/DrydockLease.php

Loading...

src/applications/drydock/storage/DrydockRepositoryOperation.php

Loading...

src/applications/drydock/worker/DrydockLeaseUpdateWorker.php

Loading...

src/applications/drydock/worker/DrydockRepositoryOperationUpdateWorker.php

Loading...