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

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

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

Branch

drydock-5

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 25706
Build 35534: arc lint + arc unit

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 51983

View Options

src/applications/drydock/worker/DrydockLeaseUpdateWorker.php

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