-
Notifications
You must be signed in to change notification settings - Fork 163
ECE links to current updated and fixed #1988
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
Conversation
🔍 Preview links for changed docs:
🔔 The preview site may take up to 3 minutes to finish building. These links will become live once it completes. |
|
||
::::{important} | ||
{{es}} 7.8 and later comes with Index Lifecycle Management (ILM) always enabled. Before upgrading to 7.8 or later, to avoid any unpredictable behavior it is important to configure hot-warm clusters on {{ece}} with ILM rather than index curation. Check [migrate to index lifecycle management](../../../manage-data/lifecycle/index-lifecycle-management.md) for existing clusters, and [configure index management](https://www.elastic.co/guide/en/cloud-enterprise/current/ece-configure-index-management.html) for new clusters. | ||
{{es}} 7.8 and later comes with Index Lifecycle Management (ILM) always enabled. Before upgrading to 7.8 or later, to avoid any unpredictable behavior it is important to configure hot-warm clusters on {{ece}} with ILM rather than index curation. Check [migrate to index lifecycle management](/manage-data/lifecycle/index-lifecycle-management/migrate-index-management.md) for existing clusters, and [configure index management](/manage-data/lifecycle/index-lifecycle-management.md) for new clusters. |
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.
The links were switched here, the migrate link was linking to manage ILM and the configure ILM was linking to a non redirected page, which we have fixed.
Now migrate goes to migrate and configure index management goes to manage ILM.
- https://www.elastic.co/guide/en/cloud-enterprise/current/ece-configure-index-management.html | ||
- https://www.elastic.co/guide/en/cloud/current/ec-configure-index-management.html |
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.
One of them was missing (ece), and I moved the EC one to this doc as it shouldn't be in the migrate doc.
--- | ||
mapped_pages: | ||
- https://www.elastic.co/guide/en/cloud-enterprise/current/ece-migrate-index-management.html | ||
- https://www.elastic.co/guide/en/cloud/current/ec-configure-index-management.html |
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.
This "migrate" doc should only have mapped pages of the ECE / EC original migrate docs, not the manage ILM docs.
You should configure all index management methods that you want your users to be able to choose from when they create their deployments from your template. You can configure index curation, index lifecycle management, or both. | ||
|
||
::::{note} | ||
Index curation has been deprecated in favor of [index lifecycle management](/manage-data/lifecycle/index-lifecycle-management.md). For {{stack}} version 6.7 and later, any deployments using index curation will be prompted to [migrate to ILM](/manage-data/lifecycle/index-lifecycle-management/migrate-index-management.md). | ||
:::: |
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.
The removed paragraph didn't have much sense, more over when index curation is deprecated and we should recommend against it.
I've added the note that can be felt a bit repetitive because in the beginning of the doc we already mention that index curation is deprecated (i think it's beneficial also to remind it just before describing both options).
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.
This changes make sense to me.
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.
Thank you, Edu! The changes make sense to me as well.
Relates to https://github.com/elastic/docs-projects/issues/494
I'm researching all links to ECE
current
doc set while finding URLs that haven't been redirected or they are incorrectly redirected.I have fixed all
cloud-enterprise/current
URLs in this repo, replacingcurrent
with3.8
when appropriate (only twice while referring to a legacy feature), or replacing them by a proper doc indocs-content
directly.I also found some little inconsisencies and I gave some extra love to some paragraphs.
Missing redirects:
https://www.elastic.co/guide/en/cloud-enterprise/current/ece-configure-index-management.html (fixed at
mapped_pages
level).https://www.elastic.co/guide/en/cloud-enterprise/current/ece-manage-apm-and-fleet.html (this one makes sense as it's a legacy feature, I've changed the links to
3.8
instead of current). No redirect should be needed (IMO), otherwise it should redirect to ECE manage Integrations Server doc.Adding comments in the code for reviewing purposes.