Skip to content

[8.5] [ML] Impose a minimum on the automatically calculated JVM size (#91732)#91742

Merged
elasticsearchmachine merged 1 commit intoelastic:8.5from
droberts195:backport/8.5/pr-91732
Nov 21, 2022
Merged

[8.5] [ML] Impose a minimum on the automatically calculated JVM size (#91732)#91742
elasticsearchmachine merged 1 commit intoelastic:8.5from
droberts195:backport/8.5/pr-91732

Conversation

@droberts195
Copy link
Copy Markdown

Backports the following commits to 8.5:

…ic#91732)

This change fixes a discrepancy that has existed for a long time
but was revealed by elastic#91694. The ML automatic node/JVM sizing code
contained a minimum node size but did not restrict the minimum
JVM size to the size that would be chosen on that minimum node
size. This could throw off calculations at small scale.

Fixes elastic#91728
@droberts195 droberts195 added :ml Machine learning >non-issue auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:ML Meta label for the ML team labels Nov 21, 2022
@elasticsearchmachine elasticsearchmachine merged commit e9f3303 into elastic:8.5 Nov 21, 2022
@droberts195 droberts195 deleted the backport/8.5/pr-91732 branch November 21, 2022 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 :ml Machine learning >non-issue Team:ML Meta label for the ML team v8.5.3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants