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

[7.x] Introduce stability description to the REST API specification (#38413) #43278

Merged
merged 1 commit into from
Jun 17, 2019

Conversation

Mpdreamz
Copy link
Member

Introduce stability to the REST API specification

(cherry picked from commit 61c34bb)

…c#38413)

* introduce state to the REST API specification

* change state over to stability

* CCR is no GA updated to stable

* SQL is now GA so marked as stable

* Introduce `internal` as state for API's, marks stable in terms of lifetime but unstable in terms of guarantees on its output format since it exposes internal representations

* make setting a wrong stability value, or not setting it at all an error that causes the YAML test suite to fail

* update spec files to be explicit about their stability state

* Document the fact that stability needs to be defined

Otherwise the YAML test runner will fail (with a nice exception message)

* address check style violations

* update rest spec unit tests to include stability

* found one more test spec file not declaring stability, made sure stability appears after documentation everywhere

* cluster.state is stable, mark response in some way to denote its a key value format that can be changed during minors

* mark data frame API's as beta

* remove internal and private as states for an API

* removed the wrong enum values in the Stability Enum in the previous commit

(cherry picked from commit 61c34bb)
@Mpdreamz Mpdreamz added :Core/Infra/REST API REST infrastructure and utilities backport v7.3.0 labels Jun 17, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@Mpdreamz Mpdreamz merged commit 8b1b9f8 into elastic:7.x Jun 17, 2019
@Mpdreamz Mpdreamz deleted the backport/7.x/spec-api-status branch June 17, 2019 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport :Core/Infra/REST API REST infrastructure and utilities v7.3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants