Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.4][ML] Validate trained model deployment queue_capacity limit #89611

Conversation

dimitris-athanasiou
Copy link
Contributor

When starting a trained model deployment, a queue is created.
If the queue_capacity is too large, it can lead to OOM and a node
crash.

This commit adds validation that the queue_capacity cannot be more
than 1M.

Closes #89555

When starting a trained model deployment, a queue is created.
If the queue_capacity is too large, it can lead to OOM and a node
crash.

This commit adds validation that the queue_capacity cannot be more
than 1M.

Closes elastic#89555
@elasticsearchmachine elasticsearchmachine added the Team:ML Meta label for the ML team label Aug 25, 2022
@elasticsearchmachine
Copy link
Collaborator

Hi @dimitris-athanasiou, I've created a changelog YAML for you.

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

Copy link
Contributor

@edsavage edsavage left a comment

Choose a reason for hiding this comment

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

LGTM

@dimitris-athanasiou dimitris-athanasiou merged commit 571d611 into elastic:8.4 Aug 25, 2022
@dimitris-athanasiou dimitris-athanasiou deleted the limit-model-deployment-queue-capacity-8_4 branch August 25, 2022 11:02
dimitris-athanasiou added a commit to dimitris-athanasiou/elasticsearch that referenced this pull request Aug 25, 2022
As it has been backported to 8.4.1 with elastic#89611
dimitris-athanasiou added a commit that referenced this pull request Aug 25, 2022
As it has been backported to 8.4.1 with #89611
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :ml Machine learning Team:ML Meta label for the ML team v8.4.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants