Skip to content

Conversation

colleenmcginnis
Copy link
Contributor

Closes https://github.com/elastic/docs-projects/issues/374

This PR adds an intro to the Data lifecycle section. It also moves "Rollup" to the bottom of this section of the table of contents since it is deprecated.

Two questions for @kilfoyle @wajihaparvez:

  • Right now https://www.elastic.co/guide/en/serverless/current/project-setting-data.html is mapped to this page, but I'm not sure that content fits here. What do you think?
  • What do you think about collapsing the "Why" part of the note about ILM not being available on serverless? I was thinking that the full note could be distracting especially for non-serverless users, but it has helpful info for serverless users.

kilfoyle
kilfoyle previously approved these changes Feb 13, 2025
Copy link
Contributor

@kilfoyle kilfoyle left a comment

Choose a reason for hiding this comment

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

LGTM! 🐇

Right now https://www.elastic.co/guide/en/serverless/current/project-setting-data.html is mapped to this page, but I'm not sure that content fits here. What do you think?

Yeah, I agree. That content is super general and doesn't belong here in the ILM docs.

What do you think about collapsing the "Why" part of the note about ILM not being available on serverless? I was thinking that the full note could be distracting especially for non-serverless users, but it has helpful info for serverless users.

I like it! ⚡

@colleenmcginnis
Copy link
Contributor Author

Right now https://www.elastic.co/guide/en/serverless/current/project-setting-data.html is mapped to this page, but I'm not sure that content fits here. What do you think?

I wasn't sure exactly where to map https://www.elastic.co/guide/en/serverless/current/project-setting-data.html if not here. I looked at how the pages linked from the page in question and it looks like it is mostly linking out to ingestion tools so I thought maybe Ingest tools overview could make sense for a redirect-only mapping.

@kilfoyle
Copy link
Contributor

I wasn't sure exactly where to map https://www.elastic.co/guide/en/serverless/current/project-setting-data.html if not here. I looked at how the pages linked from the page in question and it looks like it is mostly linking out to ingestion tools so I thought maybe Ingest tools overview could make sense for a redirect-only mapping.

Yup! That makes sense to me. Some items aren't related to "Ingest" (the Transforms section, for example), but Ingest seems to capture more than half, so that would be the closest fit.

Copy link
Contributor

@kilfoyle kilfoyle left a comment

Choose a reason for hiding this comment

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

LGTM! 🌵

@colleenmcginnis colleenmcginnis merged commit be6ced2 into elastic:main Feb 13, 2025
3 of 4 checks passed
@colleenmcginnis colleenmcginnis deleted the manage-data-lifecycle branch February 13, 2025 17:46
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.

2 participants