Skip to content

remove requirement for shards/replicas in allocation check steps#30855

Merged
talevy merged 1 commit intoelastic:index-lifecyclefrom
talevy:ilm-free-wait
May 25, 2018
Merged

remove requirement for shards/replicas in allocation check steps#30855
talevy merged 1 commit intoelastic:index-lifecyclefrom
talevy:ilm-free-wait

Conversation

@talevy
Copy link
Copy Markdown
Contributor

@talevy talevy commented May 25, 2018

As we are preparing to support policy updates/changes, we noticed
that restricting allocation wait steps with pinned replicas/shard
counts makes this difficult to continue from. For example,
as user may update or switch a policy to increase replicas. If this
is done, then the check will never pass and user intervention will
be required. If we simply remove this restriction, we still check
that the index is allocated correctly, but without depending on
the newly configured replicas setting in the policy.

As we are preparing to support policy updates/changes, we noticed
that restricting allocation wait steps with pinned replicas/shard
counts makes this difficult to continue from. For example,
as user may update or switch a policy to increase replicas. If this
is done, then the check will never pass and user intervention will
be required. If we simply remove this restriction, we still check
that the index is allocated correctly, but without depending on
the newly configured replicas setting in the policy.
@talevy talevy added the :Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. label May 25, 2018
@talevy talevy requested a review from colings86 May 25, 2018 00:49
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra

Copy link
Copy Markdown
Contributor

@colings86 colings86 left a comment

Choose a reason for hiding this comment

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

LGTM

@talevy talevy merged commit abbe8ce into elastic:index-lifecycle May 25, 2018
@talevy talevy deleted the ilm-free-wait branch May 25, 2018 14:52
jasontedor pushed a commit that referenced this pull request Aug 17, 2018
)

As we are preparing to support policy updates/changes, we noticed
that restricting allocation wait steps with pinned replicas/shard
counts makes this difficult to continue from. For example,
as user may update or switch a policy to increase replicas. If this
is done, then the check will never pass and user intervention will
be required. If we simply remove this restriction, we still check
that the index is allocated correctly, but without depending on
the newly configured replicas setting in the policy.
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants