Skip to content

Conversation

@eedugon
Copy link
Contributor

@eedugon eedugon commented Feb 3, 2026

Summary

Updating the document to set retention of logging and metrics data on ECE.

Improvements:

  • Introduction and flow within the ECE platform monitoring section.
  • Default ILM policies names identified (per https://github.com/elastic/cloud/issues/116800)
  • Procedure updated, as it wasn't completely accurate (mappings and settings were not considered) and extra steps added.
  • Added information about available index templates (not a complete list, just the most popular ones).

Closes https://github.com/elastic/docs-content-internal/issues/466
Closes https://github.com/elastic/cloud/issues/116800

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

Tool(s) and model(s) used: Cursor in auto mode

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2026

Vale Linting Results

Summary: 1 suggestion found

💡 Suggestions (1)
File Line Rule Message
deploy-manage/monitor/orchestrators/ece-monitoring-ece-set-retention.md 48 Elastic.Clone Use Clone only when referring to cloning a GitHub repository or creating a copy that is linked to the original. Often confused with 'copy' and 'duplicate'.

The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2026

@eedugon eedugon marked this pull request as ready for review February 3, 2026 22:19
@eedugon eedugon requested a review from a team as a code owner February 3, 2026 22:19
@eedugon eedugon requested review from kunisen and rseldner February 3, 2026 22:19
@eedugon eedugon changed the title ece retention logging and metrics improved ECE: customize logging and metrics retention with ILM and data streams Feb 3, 2026
Copy link
Contributor

@kunisen kunisen left a comment

Choose a reason for hiding this comment

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

I did quick check and left some comments. Mainly about making sure the user expectation on sizing, and some tech details.

@eedugon eedugon requested a review from kunisen February 4, 2026 08:51
@eedugon
Copy link
Contributor Author

eedugon commented Feb 4, 2026

@kunisen : I've made changes based on your suggestions. Really great catches.
Let me know your thoughts.

Copy link
Contributor

@kunisen kunisen left a comment

Choose a reason for hiding this comment

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

Looks really great! Thanks Edu for your time effort on this! ❤️

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! ⛵
Looks great @eedugon!

If I'm not mistaken, this new page is basically an ECE-specific version of the general procedure described in Customize built-in ILM policies in Elasticsearch, so it might be helpful to mention that somewhere with a link.

Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com>
@eedugon
Copy link
Contributor Author

eedugon commented Feb 5, 2026

Thanks for the review and improvements suggestions @davidkyle , all of them are more than welcome!

If I'm not mistaken, this new page is basically an ECE-specific version of the general procedure described in Customize built-in ILM policies in Elasticsearch, so it might be helpful to mention that somewhere with a link.

I didn't know it, thanks! I'll find the way to add it as a reference.

I finally didn't include anything about component templates and @custom examples because we are not following that path, but maybe in the future we will, if ECE managed templates provide and support them.

@eedugon eedugon enabled auto-merge (squash) February 6, 2026 08:51
@eedugon eedugon merged commit 38f0dc9 into main Feb 6, 2026
7 of 8 checks passed
@eedugon eedugon deleted the ece_logging_metrics_retention branch February 6, 2026 08:53
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.

3 participants