Skip to content
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

[DOCS] Add v3 open API spec for ml sync #141554

Merged
merged 7 commits into from
Sep 26, 2022
Merged

[DOCS] Add v3 open API spec for ml sync #141554

merged 7 commits into from
Sep 26, 2022

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Sep 23, 2022

Summary

This PR re-adds an open API v3 specification for the machine learning API based on #137242

This specification is used by #141556. If that PR is approved, we will move forward with v3 and delete the v2 specifications and output in a subsequent PR.

Relates to #137240

@lcawl lcawl added :ml docs v8.6.0 release_note:skip Skip the PR/issue when compiling release notes labels Sep 23, 2022
@lcawl lcawl marked this pull request as ready for review September 23, 2022 18:09
@lcawl lcawl requested a review from a team as a code owner September 23, 2022 18:09
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

Copy link
Contributor

@szabosteve szabosteve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two tiny suggestions, otherwise LGTM!

x-pack/plugins/ml/common/openapi/README.md Outdated Show resolved Hide resolved
x-pack/plugins/ml/common/openapi/ml_apis_v3.yaml Outdated Show resolved Hide resolved
@lcawl lcawl added v8.5.0 release_note:enhancement and removed release_note:skip Skip the PR/issue when compiling release notes labels Sep 26, 2022
@lcawl lcawl enabled auto-merge (squash) September 26, 2022 18:06
@lcawl lcawl added the skip-ci label Sep 26, 2022
@lcawl lcawl removed the skip-ci label Sep 26, 2022
@lcawl lcawl merged commit 605c958 into elastic:main Sep 26, 2022
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 26, 2022
Co-authored-by: István Zoltán Szabó <[email protected]>
(cherry picked from commit 605c958)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.5

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Sep 26, 2022
Co-authored-by: István Zoltán Szabó <[email protected]>
(cherry picked from commit 605c958)

Co-authored-by: Lisa Cawley <[email protected]>
lcawl added a commit to lcawl/kibana that referenced this pull request Sep 29, 2022
Co-authored-by: István Zoltán Szabó <[email protected]>
(cherry picked from commit 605c958)
@lcawl
Copy link
Contributor Author

lcawl commented Sep 29, 2022

💚 All backports created successfully

Status Branch Result
8.4

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@lcawl lcawl deleted the ml-v3-api branch September 29, 2022 16:56
lcawl added a commit that referenced this pull request Sep 29, 2022
Co-authored-by: István Zoltán Szabó <[email protected]>
(cherry picked from commit 605c958)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants