-
Notifications
You must be signed in to change notification settings - Fork 25.6k
[DOCS] Remove redundant ILM attributes #36808
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,24 +1,24 @@ | ||
| [role="xpack"] | ||
| [testenv="basic"] | ||
| [[ilm-get-status]] | ||
| === Get {ILM} status API | ||
| === Get {ilm} status API | ||
| ++++ | ||
| <titleabbrev>Get {ILM} status</titleabbrev> | ||
| <titleabbrev>Get {ilm} status</titleabbrev> | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I assume the change to use the long name is intentional here (as well as other titleabbrev sections) ?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, though if you prefer to have the acronym, I'm not stuck on this
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't have an opinion either way, just double checking this was intentionally. |
||
| ++++ | ||
|
|
||
| beta[] | ||
|
|
||
| Retrieves the current {ilm} status. | ||
| Retrieves the current {ilm} ({ilm-init}) status. | ||
|
|
||
| ==== Request | ||
|
|
||
| `GET /_ilm/status` | ||
|
|
||
| ==== Description | ||
|
|
||
| Returns the status of the {ILM} plugin. The `operation_mode` field in the | ||
| Returns the status of the {ilm-init} plugin. The `operation_mode` field in the | ||
| response shows one of three states: `STARTED`, `STOPPING`, | ||
| or `STOPPED`. You can change the status of the {ILM} plugin with the | ||
| or `STOPPED`. You can change the status of the {ilm-init} plugin with the | ||
| <<ilm-start, Start ILM>> and <<ilm-stop, Stop ILM>> APIs. | ||
|
|
||
| ==== Request Parameters | ||
|
|
@@ -32,7 +32,7 @@ For more information, see {stack-ov}/security-privileges.html[Security Privilege | |
|
|
||
| ==== Examples | ||
|
|
||
| The following example gets the {ILM} plugin status. | ||
| The following example gets the {ilm-init} plugin status. | ||
|
|
||
| [source,js] | ||
| -------------------------------------------------- | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -35,7 +35,7 @@ include::{docdir}/rest-api/timeoutparms.asciidoc[] | |
|
|
||
| You must have the `manage_ilm` cluster privilege to use this API. You must | ||
| also have the `manage` index privilege on all indices being managed by `policy`. | ||
| All operations executed by {Ilm} for a policy are executed as the user that | ||
| All operations executed by {ilm} for a policy are executed as the user that | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I assume the lowercase here is intentional. However, I don't see any uses of
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, I didn't see why the "I" was capitalized in this case. |
||
| put the latest version of a policy. | ||
| For more information, see {stack-ov}/security-privileges.html[Security Privileges]. | ||
|
|
||
|
|
||
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 assume the change to use the long name is intentional here (as well as other header sections) ?