Skip to content

Conversation

@stefnestor
Copy link
Contributor

👋 howdy! This may need Asciidoc reformatting. Will you kindly add in express commentary on Restore a managed Datastream or Index to also restore ILM policies as needed (via include_global_state, related doc). Otherwise, you induce ILM errors once ILM starts (and have to do a form of repeating the entire outlined procedure to get indices going through correctly.)

👋 howdy! This may need Asciidoc reformatting. Will you kindly add in express commentary on [Restore a managed Datastream or Index](https://www.elastic.co/guide/en/elasticsearch/reference/master/index-lifecycle-and-snapshots.html?edit) to also restore ILM policies as needed (via `include_global_state`). Otherwise, you induce ILM errors once ILM starts (and have to do a form of repeating the entire outlined procedure to get indices going through correctly.)
@stefnestor stefnestor added >enhancement >docs General docs changes Team:Docs Meta label for docs team labels Sep 16, 2022
@stefnestor stefnestor requested a review from debadair September 16, 2022 14:28
@github-actions
Copy link
Contributor

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 Sep 16, 2022
@csoulios csoulios added v8.6.0 and removed v8.5.0 labels Sep 21, 2022
@kingherc kingherc added v8.7.0 and removed v8.6.0 labels Nov 16, 2022
@abdonpijpelink abdonpijpelink requested review from abdonpijpelink and removed request for debadair January 17, 2023 14:06
Copy link
Contributor

@abdonpijpelink abdonpijpelink left a comment

Choose a reason for hiding this comment

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

Thanks for adding this note @stefnestor ! I think we should maybe open with this recommendation? I've reworded a bit, and fixed the broken link. Let me know what you think.

@stefnestor
Copy link
Contributor Author

Sounds great. Thanks for the feedback @abdonpijpelink !

@abdonpijpelink
Copy link
Contributor

@elasticmachine update branch

@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.6

stefnestor added a commit to stefnestor/elasticsearch that referenced this pull request Jan 23, 2023
* [+DOC] Restore policies in restoring ILM indices

👋 howdy! This may need Asciidoc reformatting. Will you kindly add in express commentary on [Restore a managed Datastream or Index](https://www.elastic.co/guide/en/elasticsearch/reference/master/index-lifecycle-and-snapshots.html?edit) to also restore ILM policies as needed (via `include_global_state`). Otherwise, you induce ILM errors once ILM starts (and have to do a form of repeating the entire outlined procedure to get indices going through correctly.)

* Apply suggestions from code review

Co-authored-by: Abdon Pijpelink <[email protected]>

Co-authored-by: Abdon Pijpelink <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
elasticsearchmachine pushed a commit that referenced this pull request Jan 23, 2023
* [+DOC] Restore policies in restoring ILM indices

👋 howdy! This may need Asciidoc reformatting. Will you kindly add in express commentary on [Restore a managed Datastream or Index](https://www.elastic.co/guide/en/elasticsearch/reference/master/index-lifecycle-and-snapshots.html?edit) to also restore ILM policies as needed (via `include_global_state`). Otherwise, you induce ILM errors once ILM starts (and have to do a form of repeating the entire outlined procedure to get indices going through correctly.)

* Apply suggestions from code review

Co-authored-by: Abdon Pijpelink <[email protected]>

Co-authored-by: Abdon Pijpelink <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>

Co-authored-by: Abdon Pijpelink <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>docs General docs changes >enhancement external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Docs Meta label for docs team v8.6.1 v8.7.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants