Skip to content

Adding a data stream lifecycle retention to templates#143713

Open
masseyke wants to merge 3 commits intoelastic:mainfrom
masseyke:add-data-stream-lifecycle-retention-to-templates
Open

Adding a data stream lifecycle retention to templates#143713
masseyke wants to merge 3 commits intoelastic:mainfrom
masseyke:add-data-stream-lifecycle-retention-to-templates

Conversation

@masseyke
Copy link
Member

@masseyke masseyke commented Mar 5, 2026

This adds a data stream lifecycle retention in templates that we deliver that is the same as their ILM retention. This is so that if the templates are used in a system where ILM is disabled, the behavior is similar.

@coderabbitai
Copy link

coderabbitai bot commented Mar 5, 2026

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (2)
  • Team:Delivery
  • Team:Search - Inference

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: c506c9d6-dfd5-4ff6-8c1b-7aba9818191d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Comment @coderabbitai help to get the list of available commands and usage tips.

@elasticsearchmachine elasticsearchmachine added the serverless-linked Added by automation, don't add manually label Mar 6, 2026
@masseyke masseyke added :StorageEngine/Data streams Data streams and their lifecycles and removed serverless-linked Added by automation, don't add manually labels Mar 6, 2026
@masseyke masseyke marked this pull request as ready for review March 6, 2026 17:31
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

Copy link
Member

@dakrone dakrone left a comment

Choose a reason for hiding this comment

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

LGTM, thanks Keith

@masseyke masseyke added the serverless-linked Added by automation, don't add manually label Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>non-issue serverless-linked Added by automation, don't add manually :StorageEngine/Data streams Data streams and their lifecycles Team:StorageEngine v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants