-
Notifications
You must be signed in to change notification settings - Fork 25.2k
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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.
There was a problem hiding this 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?
@jbaiera is there a different way to also apply it to existing data streams? |
@simitt This is what the 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. |
Closes https://github.com/elastic/apm-managed-service/issues/1541
Enable failure store to APM datastreams