-
Notifications
You must be signed in to change notification settings - Fork 62
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
Add ML Controller APIs #779
base: main
Are you sure you want to change the base?
Add ML Controller APIs #779
Conversation
Changes AnalysisCommit SHA: aa51ade API ChangesSummary
ReportThe full API changes report is available at: https://github.com/opensearch-project/opensearch-api-specification/actions/runs/12716275619/artifacts/2414931443 API Coverage
|
Looks like there's a test failure, otherwise LGTM. |
Looks like the failing test is due to the Memory Circuit Breaker flaky error https://github.com/opensearch-project/opensearch-api-specification/actions/runs/12701647488/artifacts/2410641526. I made a fix for this issue in this PR and I can rerun the test once this PR has been merged. |
Merged, rebase? |
Signed-off-by: Nathalie Jonathan <[email protected]>
Signed-off-by: Nathalie Jonathan <[email protected]>
1a18da6
to
aa51ade
Compare
Spec Test Coverage Analysis
|
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.
The logs folder got folded in, remove and add it to .gitignore?
Also check test coverage? We were at 100, we lost something.
Description
Added missing ML Controller APIs.
Issues Resolved
Part of opensearch-project/opensearch-py#867.
ML Controller APIs to add
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.