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] Refresh cached phase policy definition if possible on ne… #50941

Merged
merged 1 commit into from
Jan 13, 2020

Conversation

dakrone
Copy link
Member

@dakrone dakrone commented Jan 13, 2020

Backports the following commits to 7.x:

…tic#50820)

* Refresh cached phase policy definition if possible on new policy

There are some cases when updating a policy does not change the
structure in a significant way. In these cases, we can reread the
policy definition for any indices using the updated policy.

This commit adds this refreshing to the `TransportPutLifecycleAction`
to allow this. It allows us to do things like change the configuration
values for a particular step, even when on that step (for example,
changing the rollover criteria while on the `check-rollover-ready` step).

There are more cases where the phase definition can be reread that just
the ones checked here (for example, removing an action that has already
been passed), and those will be added in subsequent work.

Relates to elastic#48431
@dakrone dakrone added backport :Data Management/ILM+SLM Index and Snapshot lifecycle management >enhancement v7.6.0 labels Jan 13, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (:Core/Features/ILM+SLM)

@dakrone dakrone changed the title [7.x] Refresh cached phase policy definition if possible on new poli… (#50820) [7.x] Refresh cached phase policy definition if possible on ne… Jan 13, 2020
@dakrone dakrone merged commit 91689e7 into elastic:7.x Jan 13, 2020
@dakrone dakrone deleted the backport/7.x/pr-50820 branch January 13, 2020 21:31
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.

None yet

2 participants