Skip to content
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

[7.17] [Rule Registry] ignore some errors while updating index mappings (#140778) #141097

Merged
merged 1 commit into from
Sep 20, 2022

Conversation

pmuellr
Copy link
Member

@pmuellr pmuellr commented Sep 20, 2022

Backport

This will backport the following commits from main to 7.17:

Questions ?

Please refer to the Backport tool documentation

…stic#140778)

resolves elastic#139969

Changes the ResourceInstaller to ignore cases when the elasticsearch
simulateIndexTemplate() API returns an error or empty mappings, logging an
error instead. This will hopefully allow initialization to continue to set
up the alerts-as-data indices and backing resources for future indexing.

Also adds _meta: { managed: true } to the ILM policy, which should show a
warning in Kibana UX when attempting to make changes to the policy. Which
was the cause of why simulateIndexTemplate() could return empty mappings.

(cherry picked from commit 01daf31)

# Conflicts:
#	x-pack/plugins/rule_registry/server/rule_data_plugin_service/resource_installer.test.ts
#	x-pack/plugins/rule_registry/server/rule_data_plugin_service/resource_installer.ts
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@pmuellr pmuellr merged commit 8026776 into elastic:7.17 Sep 20, 2022
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.

2 participants