Skip to content

Conversation

@gwbrown
Copy link
Contributor

@gwbrown gwbrown commented Aug 18, 2021

Backports the following commits to 7.x:

…down (elastic#75750)

This PR fixes two situations where `NOT_STARTED` can appear as the shard migration status inappropriately:
1. When the node is actually shut down after having all the shards migrate away.
2. When a non-data-node is registered for shutdown.

It also adds tests to ensure these cases are handled correctly.
@gwbrown gwbrown changed the title [7.x] Fix NOT_STARTED statuses appearing inappropirately during node shutdown (#75750) Fix NOT_STARTED statuses appearing inappropirately during node shutdown (#75750) Aug 18, 2021
@gwbrown gwbrown added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Aug 18, 2021
@elasticsearchmachine elasticsearchmachine merged commit f5df6b9 into elastic:7.x Aug 18, 2021
gwbrown added a commit to gwbrown/elasticsearch that referenced this pull request Aug 18, 2021
…down (elastic#75750) (elastic#76634)

* Fix `NOT_STARTED` statuses appearing inappropirately during node shutdown (elastic#75750)

This PR fixes two situations where `NOT_STARTED` can appear as the shard migration status inappropriately:
1. When the node is actually shut down after having all the shards migrate away.
2. When a non-data-node is registered for shutdown.

It also adds tests to ensure these cases are handled correctly.

* Fix compilation for backport

* Fix compilation for backport
elasticsearchmachine pushed a commit that referenced this pull request Aug 18, 2021
…down (#75750) (#76634) (#76669)

* Fix `NOT_STARTED` statuses appearing inappropirately during node shutdown (#75750)

This PR fixes two situations where `NOT_STARTED` can appear as the shard migration status inappropriately:
1. When the node is actually shut down after having all the shards migrate away.
2. When a non-data-node is registered for shutdown.

It also adds tests to ensure these cases are handled correctly.

* Fix compilation for backport

* Fix compilation for backport
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v7.16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants