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

ILM action to wait for SLM policy execution #50454

Merged
merged 28 commits into from Jan 9, 2020

Conversation

probakowski
Copy link
Contributor

This change add new ILM action to wait for SLM policy execution to ensure that index has snapshot before deletion.

Closes #45067.

This change add new ILM action to wait for SLM policy execution to ensure that index has snapshot before deletion.

Closes elastic#45067.
@elasticmachine
Copy link
Collaborator

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

Copy link
Member

@dakrone dakrone left a comment

Choose a reason for hiding this comment

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

Thanks for working on this @probakowski! I left some comments for you.

Copy link
Member

@dakrone dakrone left a comment

Choose a reason for hiding this comment

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

LGTM, I left one comment. This also needs documentation to be added for the new action though, can you add that before merging this?

probakowski and others added 2 commits January 8, 2020 23:16
Co-Authored-By: Lee Hinman <dakrone@users.noreply.github.com>
@probakowski probakowski merged commit 060100d into elastic:master Jan 9, 2020
@probakowski probakowski deleted the ilm-wait-for-snapshot branch January 9, 2020 06:37
probakowski added a commit to probakowski/elasticsearch that referenced this pull request Jan 13, 2020
This change add new ILM action to wait for SLM policy execution to ensure that index has snapshot before deletion.

Closes elastic#45067
probakowski added a commit that referenced this pull request Jan 14, 2020
* ILM action to wait for SLM policy execution (#50454)

This change add new ILM action to wait for SLM policy execution to ensure that index has snapshot before deletion.

Closes #45067

* Fix flaky TimeSeriesLifecycleActionsIT#testWaitForSnapshot test

This change adds some randomness and cleanup step to TimeSeriesLifecycleActionsIT#testWaitForSnapshot and testWaitForSnapshotSlmExecutedBefore tests in attempt to make them stable.

Reletes to #50781

* Formatting changes

* Longer timeout

* Fix Map.of in Java8

* Unused import removed
SivagurunathanV pushed a commit to SivagurunathanV/elasticsearch that referenced this pull request Jan 23, 2020
This change add new ILM action to wait for SLM policy execution to ensure that index has snapshot before deletion.

Closes elastic#45067
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-ui (:ES-UI)

@cjcenizal cjcenizal added the Team:Deployment Management Meta label for Management Experience - Deployment Management team label Jun 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/ILM+SLM Index and Snapshot lifecycle management >feature Team:Deployment Management Meta label for Management Experience - Deployment Management team v7.6.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add an ILM action to ensure a snapshot contains the index prior to being deleted
6 participants