Skip to content

Allow SecurityIndexManager to update index mappings #68729

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

Conversation

pugnascotia
Copy link
Contributor

While backporting #67114 via #68375, I realised that there are existing
upgrade scenarios that expect the SecurityIndexManager to update index
mappings, so in the backport PR, this capability was reinstated. This
commit does the same in master.

While backporting elastic#67114 via elastic#68375, I realised that there are existing
upgrade scenarios that expect the `SecurityIndexManager` to update index
mappings, so in the backport PR, this capability was reinstated. This
commit does the same in `master`.
@pugnascotia pugnascotia requested a review from jaymode February 9, 2021 10:09
@elasticmachine elasticmachine added the Team:Security Meta label for security team label Feb 9, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security (Team:Security)

@pugnascotia pugnascotia changed the title Allow SecurityIndexManager to update mappings Allow SecurityIndexManager to update index mappings Feb 9, 2021
@pugnascotia
Copy link
Contributor Author

@elasticmachine update branch

Copy link
Member

@jaymode jaymode left a comment

Choose a reason for hiding this comment

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

LGTM

@pugnascotia pugnascotia merged commit dd2e1d7 into elastic:master Feb 9, 2021
@pugnascotia pugnascotia deleted the 61656-auto-create-security-7x-forward-port-fixes-to-master branch February 9, 2021 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>non-issue >refactoring :Security/Security Security issues without another label Team:Security Meta label for security team v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants