Skip to content

Conversation

kilfoyle
Copy link
Contributor

@kilfoyle kilfoyle commented Oct 16, 2025

Just a tiny PR to help folks find our page about applying an ILM policy to existing indices, and also reiterating the warning about not applying a policy that includes rollover.

Closes: #1567

manage-existing

@github-actions
Copy link

github-actions bot commented Oct 16, 2025

🔍 Preview links for changed docs

@kilfoyle kilfoyle marked this pull request as ready for review October 16, 2025 16:33
@kilfoyle kilfoyle requested a review from a team as a code owner October 16, 2025 16:33
Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

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

read this and was a little confused until I clicked through. if you felt like it, you might be able to clarify this further by saying when you apply the ilm policy it's matching on the specific index name or whatever the underlying reason for the disconnection is, and uplevel the "apply to template" recommendation. if you think this will all be obvious to readers at this stage feel free to ignore.

Co-authored-by: shainaraskas <58563081+shainaraskas@users.noreply.github.com>
@kilfoyle
Copy link
Contributor Author

read this and was a little confused until I clicked through. if you felt like it, you might be able to clarify this further by saying when you apply the ilm policy it's matching on the specific index name or whatever the underlying reason for the disconnection is, and uplevel the "apply to template" recommendation. if you think this will all be obvious to readers at this stage feel free to ignore.

Thanks for the suggestion. In this case I think I'll leave the warning where it is as it's a bit of an edge case, and a different scenario that just setting up an index and managing it (that is, this is specific to folks who have a ton of indices set up and haven't named them in such a way to support rollover).

@kilfoyle kilfoyle merged commit d18541e into elastic:main Oct 17, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Data lifecycle docs: Better guidance for Search users

2 participants