Skip to content

[8.5] [ML] Fix incorrect assumption about minimum ML node size (#91694)#91697

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

[8.5] [ML] Fix incorrect assumption about minimum ML node size (#91694)#91697
elasticsearchmachine merged 1 commit intoelastic:8.5from
droberts195:backport/8.5/pr-91694

Conversation

@droberts195
Copy link
Copy Markdown

Backports the following commits to 8.5:

The ML autoscaling code was making an assumption that all ML
nodes in Cloud will be at least 1GB. This is not correct. After
allowing for logging and metrics collection it is possible for
ML nodes to be smaller.

This PR updates the assumption to 0.5GB.
@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 18, 2022
@elasticsearchmachine elasticsearchmachine merged commit 5c387f8 into elastic:8.5 Nov 18, 2022
@droberts195 droberts195 deleted the backport/8.5/pr-91694 branch November 18, 2022 13:26
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