-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Index Management/ML] Fix broken retries on inference id creation #186961
[Index Management/ML] Fix broken retries on inference id creation #186961
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @sphilipse, I've tested locally and the timeout error seems to break the logic to save mappings with semantic text: on a new deployment, the request fails and it's not possible to refresh.
Adding a screen recording as well Screen.Recording.2024-06-27.at.11.22.45.mov |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ML changes LGTM
Thanks @yuliacech, looks like the timeout error check function doesn't work for this specific API. Replaced that and this should work now :) |
💚 Build Succeeded
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing the timeout issue, @sphilipse! Re-tested locally, latest changes LGTM 👍
Summary
This fixes the inference endpoint creation API being called multiple times on error. The call will often time out because downloading/deploying the model takes longer than the Kibana request timeout limit. Setting the timeout limit higher would still be fragile, so ignoring the timeout error makes more sense.
This PR also contains a few small language fixes and variable renames for clarity.