Add reference to prefer-ilm setting for apm lifeycle known bug #4501
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lahsivjar
added
backport-8.15
|
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here. |
simitt
reviewed
Nov 8, 2024
| which should be equivalent to the default ILM policies. Note that if you have a custom | ||
| lifecycle policy in place for any of the datastreams then you don't need to do anything | ||
| for those datastreams as they are not affected by this bug. Going forward, for using | ||
| custom ILM policies with new datastreams, please consult the {ref}/apm-ilm-how-to.html[updated guide]. |
There was a problem hiding this comment.
Why do you think this belongs here? If I have a custom ILM policy I don't even read this line due to
The issue also does not occur if a custom ILM policy is configured using a custom component template.
There was a problem hiding this comment.
Took a stab at updating this in a different way to avoid confusion. PTAL.
There was a problem hiding this comment.
This part looks good to me now - are you planning on also adding the prefer-ilm section to this PR or a separate one? (The PR description is not updated)
There was a problem hiding this comment.
Created a separate PR for that: https://github.com/elastic/observability-docs/pull/4505/files
Co-authored-by: Colleen McGinnis <[email protected]>
|
Added two commits from an open PR (#4506) that also makes updates to this file. |
|
I made a few small changes to the text. Please take a look and let me know if this still looks okay. If so, we'll get this merged. Thanks! |
lahsivjar
commented
Nov 8, 2024
|
Thanks @bmorelli25 , a minor comment - rest LGTM! |
Co-authored-by: Vishal Raj <[email protected]>
bmorelli25
approved these changes
Nov 8, 2024
mergify bot
pushed a commit
that referenced
this pull request
Nov 8, 2024
* Add reference to prefer-ilm setting for apm lifeycle known bug * Refactor * APM known issue correction * Update docs/en/observability/apm/known-issues.asciidoc Co-authored-by: Colleen McGinnis <[email protected]> * clean up text * Update docs/en/observability/apm/known-issues.asciidoc Co-authored-by: Vishal Raj <[email protected]> --------- Co-authored-by: Francesco Gualazzi <[email protected]> Co-authored-by: Brandon Morelli <[email protected]> Co-authored-by: Colleen McGinnis <[email protected]> (cherry picked from commit c576b14)
10 tasks
mergify bot
pushed a commit
that referenced
this pull request
Nov 8, 2024
* Add reference to prefer-ilm setting for apm lifeycle known bug * Refactor * APM known issue correction * Update docs/en/observability/apm/known-issues.asciidoc Co-authored-by: Colleen McGinnis <[email protected]> * clean up text * Update docs/en/observability/apm/known-issues.asciidoc Co-authored-by: Vishal Raj <[email protected]> --------- Co-authored-by: Francesco Gualazzi <[email protected]> Co-authored-by: Brandon Morelli <[email protected]> Co-authored-by: Colleen McGinnis <[email protected]> (cherry picked from commit c576b14)
mergify bot
pushed a commit
that referenced
this pull request
Nov 8, 2024
* Add reference to prefer-ilm setting for apm lifeycle known bug * Refactor * APM known issue correction * Update docs/en/observability/apm/known-issues.asciidoc Co-authored-by: Colleen McGinnis <[email protected]> * clean up text * Update docs/en/observability/apm/known-issues.asciidoc Co-authored-by: Vishal Raj <[email protected]> --------- Co-authored-by: Francesco Gualazzi <[email protected]> Co-authored-by: Brandon Morelli <[email protected]> Co-authored-by: Colleen McGinnis <[email protected]> (cherry picked from commit c576b14)
This was referenced Nov 8, 2024
bmorelli25
pushed a commit
that referenced
this pull request
Nov 8, 2024
#4515) * Add reference to prefer-ilm setting for apm lifeycle known bug * Refactor * APM known issue correction * Update docs/en/observability/apm/known-issues.asciidoc Co-authored-by: Colleen McGinnis <[email protected]> * clean up text * Update docs/en/observability/apm/known-issues.asciidoc Co-authored-by: Vishal Raj <[email protected]> --------- Co-authored-by: Francesco Gualazzi <[email protected]> Co-authored-by: Brandon Morelli <[email protected]> Co-authored-by: Colleen McGinnis <[email protected]> (cherry picked from commit c576b14) Co-authored-by: Vishal Raj <[email protected]>
bmorelli25
pushed a commit
that referenced
this pull request
Nov 8, 2024
#4514) * Add reference to prefer-ilm setting for apm lifeycle known bug * Refactor * APM known issue correction * Update docs/en/observability/apm/known-issues.asciidoc Co-authored-by: Colleen McGinnis <[email protected]> * clean up text * Update docs/en/observability/apm/known-issues.asciidoc Co-authored-by: Vishal Raj <[email protected]> --------- Co-authored-by: Francesco Gualazzi <[email protected]> Co-authored-by: Brandon Morelli <[email protected]> Co-authored-by: Colleen McGinnis <[email protected]> (cherry picked from commit c576b14) Co-authored-by: Vishal Raj <[email protected]>
bmorelli25
pushed a commit
that referenced
this pull request
Nov 8, 2024
#4516) * Add reference to prefer-ilm setting for apm lifeycle known bug * Refactor * APM known issue correction * Update docs/en/observability/apm/known-issues.asciidoc Co-authored-by: Colleen McGinnis <[email protected]> * clean up text * Update docs/en/observability/apm/known-issues.asciidoc Co-authored-by: Vishal Raj <[email protected]> --------- Co-authored-by: Francesco Gualazzi <[email protected]> Co-authored-by: Brandon Morelli <[email protected]> Co-authored-by: Colleen McGinnis <[email protected]> (cherry picked from commit c576b14) Co-authored-by: Vishal Raj <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Clarify that one of the suggested fix for the unmanaged indices bug introduced in 8.15.0 and fixed in 8.15.1 should only be applied for datatreams with default policies (as they are the only ones affected). If the suggested fix was applied to a datastream using custom ILM policy then the
prefer_ilmsetting needs to be added to the custom ILM policy.Documentation sets edited in this PR
Check all that apply.
docs/en/observability/*)docs/en/serverless/*)docs/en/integrations/*)Related issue
Closes #
Checklist
Follow-up tasks
Select one.