Skip to content

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

Merged
dimitris-athanasiou merged 3 commits intoelastic:8.4from
dimitris-athanasiou:limit-model-deployment-queue-capacity-8_4
Aug 25, 2022
Merged

[8.4][ML] Validate trained model deployment queue_capacity limit#89611
dimitris-athanasiou merged 3 commits intoelastic:8.4from
dimitris-athanasiou:limit-model-deployment-queue-capacity-8_4

Conversation

@dimitris-athanasiou
Copy link
Copy Markdown
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
Copy Markdown
Collaborator

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

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
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