In Drydock, yield for reclaiming resources in the "released" state
ClosedPublic
Actions

Authored by epriestley on May 9 2022, 5:45 PM.

Tags

None

Referenced Files

	F15429561: D21809.diff
	Mon, Mar 24, 2:57 AM

	F15420595: D21809.diff
	Fri, Mar 21, 3:16 PM

	F15408799: D21809.diff
	Wed, Mar 19, 1:24 AM

	F15392698: D21809.id51983.diff
	Sat, Mar 15, 5:06 PM

	F15391811: D21809.id51984.diff
	Sat, Mar 15, 11:46 AM

	F15391809: D21809.id51983.diff
	Sat, Mar 15, 11:45 AM

	F15391807: D21809.id.diff
	Sat, Mar 15, 11:45 AM

	F15383448: D21809.diff
	Fri, Mar 14, 4:44 PM

View All Files

Subscribers

Restricted Owners Package

Details

Reviewers: None
Maniphest Tasks: T13677: Drydock may grow resource pools too cautiously
Commits: rPa640a4a4930f: In Drydock, yield for reclaiming resources in the "released" state

Summary

Ref T13677. This was an accidental change in D21807: when reclaiming a resource, wait for it to be completely destroyed before allowing a lease to reclaim another resource.

Test Plan

Reverts accidental behavior change in D21807.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.May 9 2022, 5:45 PM

Owners added a subscriber: Restricted Owners Package.May 9 2022, 5:45 PM

epriestley requested review of this revision.May 9 2022, 5:45 PM

Harbormaster completed remote builds in B25706: Diff 51983.May 9 2022, 5:45 PM

This revision was not accepted when it landed; it landed in state Needs Review.May 9 2022, 5:45 PM

Closed by commit rPa640a4a4930f: In Drydock, yield for reclaiming resources in the "released" state (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPa640a4a4930f: In Drydock, yield for reclaiming resources in the "released" state.

Revision Contents
Changeset List

Path

Size

src/

applications/

drydock/

worker/

DrydockLeaseUpdateWorker.php

1 line

Diff 51984

View Options

src/applications/drydock/worker/DrydockLeaseUpdateWorker.php

In Drydock, yield for reclaiming resources in the "released" stateClosedPublicActions