Page MenuHomePhabricator

Remove the "25% of active pool" growth rate throttle from Drydock
ClosedPublic

Authored by epriestley on May 9 2022, 5:30 PM.
Tags
None
Referenced Files
F13087578: D21808.diff
Thu, Apr 25, 1:00 AM
Unknown Object (File)
Sat, Apr 20, 5:56 PM
Unknown Object (File)
Sat, Apr 20, 4:55 PM
Unknown Object (File)
Fri, Apr 19, 8:53 AM
Unknown Object (File)
Mon, Apr 15, 10:45 PM
Unknown Object (File)
Mon, Apr 15, 8:49 PM
Unknown Object (File)
Tue, Apr 9, 1:00 PM
Unknown Object (File)
Tue, Apr 9, 12:49 PM
Subscribers
Restricted Owners Package

Details

Summary

Ref T13677. Drydock has a hard-coded and fairly arbitrary limit which prevents a resource pool from growing more than 25% at once.

This is vaguely reasonable for resources which allocate quickly, but suffocating for slower resources. It's also wholly arbitrary, and the "one per lease" limit introduced in D21807 should do a better job of covering the same ground while generally being more reasonable, expected, and predictable.

Test Plan

Ran Drydock allocations without the throttle, saw faster pool growth.

Diff Detail

Repository
rP Phabricator
Branch
drydock-4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25705
Build 35533: arc lint + arc unit

Event Timeline

Owners added a subscriber: Restricted Owners Package.May 9 2022, 5:30 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 9 2022, 5:40 PM
This revision was automatically updated to reflect the committed changes.