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

HLRC: Get Deprecation Info API #36279

Merged
merged 3 commits into from Dec 7, 2018

Conversation

hub-cap
Copy link
Contributor

@hub-cap hub-cap commented Dec 5, 2018

This commit adds the Get Deprecation Info API and associated
documentation.

Relates #29827

This commit adds the Get Deprecation Info API and associated
documentation.

Relates elastic#29827
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features

Copy link
Contributor

@gwbrown gwbrown left a comment

Choose a reason for hiding this comment

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

Left a few comments as I've been hip-deep in the server side of this code for the last few days.

static Request getDeprecationInfo(DeprecationInfoRequest deprecationInfoRequest) {
String endpoint = new RequestConverters.EndpointBuilder()
.addCommaSeparatedPathParts(deprecationInfoRequest.getIndices())
.addPathPartAsIs("_xpack", "migration", "deprecations")
Copy link
Contributor

Choose a reason for hiding this comment

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

In 7.0 this should use _migration/deprecations as of #35976, although in 6.x it will have to use _xpack.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do u think its worth me doing that in a new PR after this so i can backport it back, similar to the way we did the other PRs?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

lol i guess i dont have a choice!

   > Warnings: [[GET /{index}/_xpack/migration/deprecations] is deprecated! Use [GET /{index}/_migration/deprecations] instead.]


public enum Level {
NONE,
INFO,
Copy link
Contributor

Choose a reason for hiding this comment

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

Soon (likely tomorrow) I'll have a PR removing NONE and INFO from this list server-side - whichever merges last will have to remove them from this list too.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

++ pls link the PR when u have it

Copy link
Contributor

Choose a reason for hiding this comment

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

Aforementioned PR: #36326

Copy link
Contributor Author

Choose a reason for hiding this comment

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

looks like i won ;)

[id="{upid}-{api}-response"]
==== Get Deprecation Info Response

The returned +{response}+ contains the cluster, index and node level deprecations.
Copy link
Contributor

Choose a reason for hiding this comment

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

This is awkward and not very clear, do you think we could reword it to something like:

Suggested change
The returned +{response}+ contains the cluster, index and node level deprecations.
The returned +{response}+ contains information about deprecated features currently in use at the cluster, node, and index level.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

woah what is this black magic commit suggestion thing? I kinda want to click it just to have played around with it!

Copy link
Contributor

@gwbrown gwbrown left a comment

Choose a reason for hiding this comment

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

LGTM

@hub-cap hub-cap merged commit ca09936 into elastic:master Dec 7, 2018
hub-cap added a commit that referenced this pull request Dec 7, 2018
This commit adds the Get Deprecation Info API and associated
documentation.

Note: This commit is different from master as it adds back the
_xpack portion of the API.

Relates #29827
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.

None yet

4 participants