Skip to content

[8.18] [ML] Fix model deployment check in file uploader (#209585)#210644

Merged
kibanamachine merged 1 commit intoelastic:8.18from
kibanamachine:backport/8.18/pr-209585
Feb 11, 2025
Merged

[8.18] [ML] Fix model deployment check in file uploader (#209585)#210644
kibanamachine merged 1 commit intoelastic:8.18from
kibanamachine:backport/8.18/pr-209585

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.18:

Questions ?

Please refer to the Backport tool documentation

Adding an additional check to see if the selected model has been
deployed before continuing with the upload.
The `num_allocations` count can occasionally be incorrect, so this also
checks to see if the initial call to the inference api has returned with
no error. If it has, the model must be deployed.

To test:
Manually deploy both `.elser_model_2` and `.multilingual-e5-small` in
ML's trained models page.

In dev tools call:
```
POST /_inference/.multilingual-e5-small-elasticsearch
{
"input": ""
}
```
then call:
```
POST /_inference/.elser-2-elasticsearch
{
"input": ""
}
```
Once both models have been deployed, check the `num_allocations` of
`.elser-2-elasticsearch`
```
GET /_inference/.elser-2-elasticsearch
```

It should be `0` even though the model has been deployed.
If it isn't `0`. wait around 15mins for both models to scale down then
repeat the dev tools commands.
Call this to check `num_allocations ` in both models.
```
GET /_inference/_all
```

(cherry picked from commit 06801d8)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Feb 11, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) February 11, 2025 17:20
@kibanamachine kibanamachine merged commit 497b2cd into elastic:8.18 Feb 11, 2025
3 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
dataVisualizer 721.0KB 721.2KB +190.0B

cc @jgowdyelastic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants