Skip to content

Conversation

@rbtr
Copy link
Collaborator

@rbtr rbtr commented Sep 9, 2022

Signed-off-by: Evan Baker rbtr@users.noreply.github.com

Reason for Change:

Currently the scale-up logic blindly adds +1 batch to whatever the current request is, meaning that if the current request is not aligned to the batch, the post scale-up pool will still not be aligned to the batch. During scale down, we calculate the skew from the batch and use that to realign the pool to a multiple of the batch.
This change copies the mod arithmetic from the scale-down so that we realign to the next greater multiple of the batch size during a scale-up. This may be within the threshold that triggers another scale-up, ie:
a pool of batch 16 with 15 IPs and 14 assignments will scale up to -> 16, which is less than the minimum free IPs, so it will scale up again to 32 on the next tick. This behavior is seen when coming out of a constrained subnet state, when the batch would have been temporarily adjusted to 1 and thus the requested IP count could be anything.

Issue Fixed:

Requirements:

Notes:

@rbtr rbtr requested a review from a team as a code owner September 9, 2022 00:07
@rbtr rbtr requested review from rsagasthya and removed request for a team September 9, 2022 00:07
@rbtr rbtr self-assigned this Sep 9, 2022
@rbtr rbtr added cns Related to CNS. swift Related to SWIFT networking. fix Fixes something. labels Sep 9, 2022
@rbtr rbtr requested a review from thatmattlong September 9, 2022 00:07
@rbtr rbtr force-pushed the fix/batch-scale-up branch from 488d775 to ff10bdd Compare September 9, 2022 00:29
@rbtr rbtr changed the title align pool to batch during scale up Align pool to batch during scale up Sep 12, 2022
rsagasthya
rsagasthya previously approved these changes Sep 12, 2022
Copy link
Contributor

@rsagasthya rsagasthya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving the changes. But, looks like the branch is out of date.

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
@rbtr rbtr force-pushed the fix/batch-scale-up branch from 476aba8 to 349bd92 Compare September 14, 2022 21:50
@rbtr rbtr merged commit bd7ee76 into Azure:master Sep 15, 2022
@rbtr rbtr deleted the fix/batch-scale-up branch September 15, 2022 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cns Related to CNS. fix Fixes something. swift Related to SWIFT networking.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants