-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Rule Registry] Updating index.mapping.total_fields.limit
index setting on concrete backing indices
#143409
[Rule Registry] Updating index.mapping.total_fields.limit
index setting on concrete backing indices
#143409
Conversation
index.mapping.total_fields.limit
index setting on concrete backing indices
Pinging @elastic/response-ops (Team:ResponseOps) |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
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.
Actionable Observability changes LGTM
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]
History
To update your PR or re-run it, just comment with: cc @ymao1 |
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
Summary
Upgrade testing on the QA cluster revealed that this PR to add the ECS component template to the observability alerts as data index templates will cause errors during rule registry initialization when the mappings are updated for existing concrete indices for 7.17 that have a total field limit of 1000. While we specify a higher limit in the index template specification, those changes are not applied to existing indices, instead they will apply when the indices roll over. This PR updates this specific index setting for existing concrete indices.
To Verify