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

Update ilm-searchable-snapshot.asciidoc #95255

Merged
merged 2 commits into from
Apr 20, 2023
Merged

Update ilm-searchable-snapshot.asciidoc #95255

merged 2 commits into from
Apr 20, 2023

Conversation

stefnestor
Copy link
Contributor

👋🏼 howdy, team! Can we explicate that ILM Searchable Snapshot retention (doc) is not effected by SLM policies? Advanced users can deduce this via the SLM policy doc, but I'd like to make it easier / less time consuming for users to confirm. TIA!

👋🏼 howdy, team! Can we explicate that ILM Searchable Snapshot retention ([doc](https://www.elastic.co/guide/en/elasticsearch/reference/master/ilm-searchable-snapshot.html)) is not effected by SLM policies? TIA!
@stefnestor stefnestor added >docs General docs changes Team:Docs Meta label for docs team labels Apr 14, 2023
@github-actions
Copy link

Documentation preview:

@elasticsearchmachine
Copy link
Collaborator

@stefnestor please enable the option "Allow edits and access to secrets by maintainers" on your PR. For more information, see the documentation.

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@elasticsearchmachine elasticsearchmachine added the external-contributor Pull request authored by a developer outside the Elasticsearch team label Apr 14, 2023
@amyjtechwriter amyjtechwriter self-requested a review April 17, 2023 11:58
Copy link
Contributor

@amyjtechwriter amyjtechwriter left a comment

Choose a reason for hiding this comment

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

LGTM - I've just left a comment which just expands the abbreviations but it can be ignored.

@stefnestor
Copy link
Contributor Author

👋🏼 @amyjtechwriter thanks for the review! I like your update but don't know how to fix my original PR w/o just approving any changes you make to merge. Can you help me apply your comment? (I don't have Elasticsearch Github-local, I use the ?edit button on the doc URL so can't change-things-other-than-accept-updates after creating the original PR.) 🙏🏼

@amyjtechwriter
Copy link
Contributor

Hey @stefnestor, no worries. The comment has been applied now.

@stefnestor stefnestor merged commit dd968f5 into main Apr 20, 2023
1 check passed
@stefnestor stefnestor deleted the stefnestor-patch-1 branch April 20, 2023 13:03
@stefnestor
Copy link
Contributor Author

Huzzah, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Docs Meta label for docs team v8.8.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants