-
Notifications
You must be signed in to change notification settings - Fork 64
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
Added _ml/agents
and ml/connectors
specs
#585
Conversation
…with test Signed-off-by: Junwei Dai <[email protected]>
Signed-off-by: Junwei Dai <[email protected]>
Changes AnalysisCommit SHA: 030944a API ChangesSummary
ReportThe full API changes report is available at: https://github.com/opensearch-project/opensearch-api-specification/actions/runs/10945790110/artifacts/1954430061 API Coverage
|
Signed-off-by: Junwei Dai <[email protected]>
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.
LGTM from an API standpoint!
required: | ||
- actions | ||
- credential | ||
- description | ||
- name | ||
- parameters | ||
- protocol | ||
- version |
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.
This is exactly the data we're going to want to pull programmatically to validate our templates. Glad to see it being added!
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.
LGTM, small stuff for the broken link in CHANGELOG (missing a )
).
Co-authored-by: Daniel (dB.) Doubrovkine <[email protected]> Signed-off-by: Junwei Dai <[email protected]>
Signed-off-by: Junwei Dai <[email protected]>
Signed-off-by: Junwei Dai <[email protected]>
* Add Ml api Spec , /connector/_register and /connector/{connector_id} with test Signed-off-by: Junwei Dai <[email protected]> * Add Ml api Spec , /agents/_register and /agents/{agent_id} with test Signed-off-by: Junwei Dai <[email protected]> * fix: agent_id error Signed-off-by: Junwei Dai <[email protected]> * Update CHANGELOG.md Co-authored-by: Daniel (dB.) Doubrovkine <[email protected]> Signed-off-by: Junwei Dai <[email protected]> * Readded empty line Signed-off-by: Junwei Dai <[email protected]> --------- Signed-off-by: Junwei Dai <[email protected]> Signed-off-by: Junwei Dai <[email protected]> Co-authored-by: Junwei Dai <[email protected]> Co-authored-by: Daniel (dB.) Doubrovkine <[email protected]>
Description
This PR documents a subset of the ML Commons APIs consumed by the Flow Framework client.
Previously covered API specs (in other PRs):
POST /_plugins/_ml/model_groups/_register
POST /_plugins/_ml/models/_register
DELETE /_plugins/_ml/models/{model_id}
POST /_plugins/_ml/models/{model_id}/_deploy
POST /_plugins/_ml/models/{model_id}/_undeploy
New API specs added in this PR:
POST /_plugins/_ml/agents/_register
DELETE /_plugins/_ml/agents/{agent_id}
POST /_plugins/_ml/connectors/_create
DELETE /_plugins/_ml/connectors/{connector_id}
Issues Resolved
Part of #228
Task 3 of #833
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.