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.x] [ILM] respect overwrite config in setup (#14741) #14819

Merged
merged 2 commits into from
Nov 28, 2019

Conversation

simitt
Copy link
Contributor

@simitt simitt commented Nov 27, 2019

Backports the following commits to 7.x:

Respect `setup.ilm.overwrite` configuration when running setup
cmd. Print warning if overwrite is disabled on setup cmd.

fixes elastic#14735
@@ -24,6 +24,8 @@ https://github.com/elastic/beats/compare/v7.0.0-alpha2...master[Check the HEAD d
- Adding new `Enterprise` license type to the licenser. {issue}14246[14246]
- Fix memory leak in kubernetes autodiscover provider and add_kubernetes_metadata processor happening when pods are terminated without sending a delete event. {pull}14259[14259]
- Allow Metricbeat's beat module to read monitoring information over a named pipe or unix domain socket. {pull}14558[14558]
- Remove version information from default ILM policy for improved upgrade experience on custom policies. {pull}14745[14745]
- Running `setup` cmd respects `setup.ilm.overwrite` setting for improved support of custom policies. {pull}14741[14741]
Copy link

Choose a reason for hiding this comment

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

Can you fix the changelog please?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Could only be resolved after merging the other backport. Fixed now.

@simitt simitt merged commit dbbceff into elastic:7.x Nov 28, 2019
@simitt simitt deleted the backport/7.x/pr-14741 branch August 4, 2020 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants