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
[ILM] fix rest-api-spec doc links #35478
Conversation
Pinging @elastic/es-core-infra |
@@ -1,6 +1,6 @@ | |||
{ | |||
"ilm.delete_lifecycle": { | |||
"documentation": "http://www.elastic.co/guide/en/index_lifecycle/current/index_lifecycle.html", | |||
"documentation": "https://www.elastic.co/guide/en/elasticsearch/reference/current/ilm-delete-lifecycle.html", |
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.
These links are wrong I think (at least they 404) as the API docs are not in the guide book. I think they should be https://www.elastic.co/guide/en/elasticsearch/reference/current/ilm-delete-policy.html
, which still 404 because the current version is 6.4 right now but if you replace current
with master
or 6.x
they work so they should be ok when current becomes 6.6
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.
I'm not sure what is being recommend here. I think the intention of using current
here is to prevent the need to update this all the time between versions, and the assumption is these are valid in the branch that it is merged into
the links to the docs from the rest-api-spec were stale. Now that the API docs
for ILM exist, these are updated.