Skip to content

Enable failure store for log data streams#131261

Merged
eyalkoren merged 24 commits intoelastic:mainfrom
eyalkoren:logs-enable-faliure-store
Jul 31, 2025
Merged

Enable failure store for log data streams#131261
eyalkoren merged 24 commits intoelastic:mainfrom
eyalkoren:logs-enable-faliure-store

Conversation

@eyalkoren
Copy link
Copy Markdown
Contributor

@eyalkoren eyalkoren commented Jul 15, 2025

Closes #131105
Now let's see which tests this breaks 🙂

@eyalkoren eyalkoren requested a review from felixbarny July 15, 2025 03:38
@eyalkoren eyalkoren self-assigned this Jul 15, 2025
@eyalkoren eyalkoren added >feature :StorageEngine/Data streams Data streams and their lifecycles v9.2.0 labels Jul 15, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Data Management (obsolete) DO NOT USE. This team no longer exists. label Jul 15, 2025
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Hi @eyalkoren, I've created a changelog YAML for you.

@elasticsearchmachine elasticsearchmachine added the serverless-linked Added by automation, don't add manually label Jul 15, 2025
Copy link
Copy Markdown
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.

I believe this needs to have the StackTemplateRegistry#REGISTRY_VERSION number bumped, but otherwise looks good to me. I left a couple of minor comments, but looks fine after CI passes.

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Hi @eyalkoren, I've updated the changelog YAML for you. Note that since this PR is labelled release highlight, you need to update the changelog YAML to fill out the extended information sections.

@eyalkoren eyalkoren requested a review from mosche July 29, 2025 07:21
@eyalkoren
Copy link
Copy Markdown
Contributor Author

@mosche please see 10af65e, which includes the change we discussed. Thanks for your assistance! 🙏

Copy link
Copy Markdown
Contributor

@mosche mosche 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 switching to test features, that part lgtm

@eyalkoren eyalkoren merged commit 3eab53c into elastic:main Jul 31, 2025
33 checks passed
@eyalkoren eyalkoren deleted the logs-enable-faliure-store branch July 31, 2025 05:10
afoucret pushed a commit to afoucret/elasticsearch that referenced this pull request Jul 31, 2025
smalyshev pushed a commit to smalyshev/elasticsearch that referenced this pull request Jul 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>feature release highlight serverless-linked Added by automation, don't add manually :StorageEngine/Data streams Data streams and their lifecycles Team:Data Management (obsolete) DO NOT USE. This team no longer exists. v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable failure store for logs by default

6 participants