Skip to content

Enable failure store to APM datastreams #125672

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rubvs
Copy link
Contributor

@rubvs rubvs commented Mar 26, 2025

Closes https://github.com/elastic/apm-managed-service/issues/1541

Enable failure store to APM datastreams

Copy link
Member

@jbaiera jbaiera left a comment

Choose a reason for hiding this comment

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

LGTM, though this will likely need to wait until after the feature is out from behind the feature flag to be merged.

Copy link
Member

@jbaiera jbaiera left a comment

Choose a reason for hiding this comment

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

This won't handle APM data streams that already exist when applied, is that something that needs to be considered as well for this rollout?

@rubvs
Copy link
Contributor Author

rubvs commented Apr 2, 2025

This won't handle APM data streams that already exist when applied, is that something that needs to be considered as well for this rollout?

@1pkg and @simitt can you please give your input on this?

@simitt
Copy link
Contributor

simitt commented Apr 15, 2025

@jbaiera is there a different way to also apply it to existing data streams?

@jbaiera
Copy link
Member

jbaiera commented Apr 16, 2025

is there a different way to also apply it to existing data streams?

@simitt This is what the data_streams.failure_store.enabled cluster setting is for. It takes a list of index wildcard patterns as a value, and any data streams that match the patterns will have failure store enabled, as long as those data streams do not have failure store explicitly enabled/disabled. It's effectively selecting the data stream names that will have the feature enabled by default if not currently specified.

We'll need to use both the template and the cluster setting as part of the rollout. The template to enable the failure store on all data streams going forward, and the cluster setting to pick up any existing data streams from before the template update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants