Skip to content

ILM retryable async action steps (#50522)#50591

Merged
andreidan merged 2 commits intoelastic:7.xfrom
andreidan:backport/7x-retryable-rolloverstep
Jan 3, 2020
Merged

ILM retryable async action steps (#50522)#50591
andreidan merged 2 commits intoelastic:7.xfrom
andreidan:backport/7x-retryable-rolloverstep

Conversation

@andreidan
Copy link
Copy Markdown
Contributor

This adds support for retrying AsyncActionSteps by triggering the async
step after ILM was moved back on the failed step (the async step we'll
be attempting to run after the cluster state reflects ILM being moved
back on the failed step).

This also marks the RolloverStep as retryable and adds an integration
test where the RolloverStep is failing to execute as the rolled over
index already exists to test that the async action RolloverStep is
retried until the rolled over index is deleted.

(cherry picked from commit 8bee5f4)
Signed-off-by: Andrei Dan andrei.dan@elastic.co

Backport of #50522

This adds support for retrying AsyncActionSteps by triggering the async
step after ILM was moved back on the failed step (the async step we'll
be attempting to run after the cluster state reflects ILM being moved
back on the failed step).

This also marks the RolloverStep as retryable and adds an integration
test where the RolloverStep is failing to execute as the rolled over
index already exists to test that the async action RolloverStep is
retried until the rolled over index is deleted.

(cherry picked from commit 8bee5f4)
Signed-off-by: Andrei Dan <andrei.dan@elastic.co>
@andreidan
Copy link
Copy Markdown
Contributor Author

@elasticmachine update branch

@andreidan andreidan merged commit 3c971f2 into elastic:7.x Jan 3, 2020
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