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

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

Merged
merged 1 commit into from
Aug 18, 2021

Conversation

gwbrown
Copy link
Contributor

@gwbrown gwbrown commented Aug 18, 2021

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.

…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
@gwbrown gwbrown added backport v7.15.0 auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Aug 18, 2021
@elasticsearchmachine elasticsearchmachine merged commit 3de609e into elastic:7.15 Aug 18, 2021
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.15.0 v7.15.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants