Skip to content

ILM open/close steps are noop if idx is open/close#48614

Merged
andreidan merged 1 commit intoelastic:masterfrom
andreidan:ilm-open-close-follower-steps
Oct 29, 2019
Merged

ILM open/close steps are noop if idx is open/close#48614
andreidan merged 1 commit intoelastic:masterfrom
andreidan:ilm-open-close-follower-steps

Conversation

@andreidan
Copy link
Copy Markdown
Contributor

The open and close follower steps didn't check if the index is open,
closed respectively, before executing the open/close request.
This changes the steps to check the index state and only perform the
open/close operation if the index is not already open/closed.

This avoids a rather expensive operation and improves the resiliency of
the steps as a possible point of failure is avoided.

The open and close follower steps didn't check if the index is open,
closed respectively, before executing the open/close request.
This changes the steps to check the index state and only perform the
open/close operation if the index is not already open/closed.
@andreidan andreidan added >enhancement :Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. v8.0.0 v7.6.0 labels Oct 29, 2019
@elasticmachine
Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
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

@andreidan andreidan merged commit 7923145 into elastic:master Oct 29, 2019
andreidan added a commit to andreidan/elasticsearch that referenced this pull request Oct 29, 2019
The open and close follower steps didn't check if the index is open,
closed respectively, before executing the open/close request.
This changes the steps to check the index state and only perform the
open/close operation if the index is not already open/closed.
andreidan added a commit that referenced this pull request Oct 29, 2019
The open and close follower steps didn't check if the index is open,
closed respectively, before executing the open/close request.
This changes the steps to check the index state and only perform the
open/close operation if the index is not already open/closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. >enhancement v7.6.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants