Skip to content

Conversation

valeriy42
Copy link
Contributor

Backport

This will backport the following commits from main to 8.18:

Questions ?

Please refer to the Backport tool documentation

…ic#132170)

A follow-up to elastic#131990. This PR ensures that only assigned allocations and not current allocations are used in the memory requirements calculation in AssignmentPlan.

This change led to the simplification of the code in ZoneAwareAssignmentPlanner and TrainedModelRebalancer.

This PR also improves readability by adding comments, code documentation, renaming variables, and making the flow of if statements more straightforward.

Marking is a non-issue since the bug was already documented in elastic#131990.

(cherry picked from commit 80c47f3)
@valeriy42 valeriy42 added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Aug 1, 2025
@elasticsearchmachine elasticsearchmachine merged commit 727d5e7 into elastic:8.18 Aug 1, 2025
16 checks passed
@valeriy42 valeriy42 deleted the backport/8.18/pr-132170 branch August 1, 2025 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v8.18.5
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants