Add ml inference processor and APIs to console#53618
Add ml inference processor and APIs to console#53618benwtrent merged 1 commit intoelastic:masterfrom
Conversation
|
Pinging @elastic/es-ui (Team:Elasticsearch UI) |
💚 Build SucceededTo update your PR or re-run it, just comment with: |
cjcenizal
left a comment
There was a problem hiding this comment.
Tested locally and the autocomplete LGTM! Processor autocomplete worked well too:
However, it looks like when I hit any of these endpoints I get a 400 error. Am I doing something wrong? Or have I found a bug?
|
Were these endpoints added to ES very recently? If so then the 400 errors may be due to the pinned ES version being out of date, which is being addressed by #53645. |
|
@cjcenizal these endpoints are in 8.0.0 and 7.6.0 :). As for when the PR for the endpoints was merged, it was in the last month or so. I have verified that the endpoint works on master. Do you know the pinned commit hash? I could double check before merge if you are concerned. |
|
Seems to work for me :). |
Adds new Machine Learning Inference processor to Kibana Console. Also adds new Inference APIs generated from ES specs.
Adds new Machine Learning Inference processor to Kibana Console. Also adds new Inference APIs generated from ES specs.
Summary
Adds new Machine Learning inference processor to Kibana Console.
Also adds new inference APIs generated from ES specs.
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.[ ] This was checked for cross-browser compatibility, including a check against IE11[ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support[ ] Documentation](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#writing-documentation) was added for features that require explanation or tutorials[ ] Unit or functional tests were updated or added to match the most common scenarios[ ] This was checked for keyboard-only and screenreader accessibilityFor maintainers