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

Skip ancient closed indices in desired balance #91765

Conversation

DaveCTurner
Copy link
Contributor

This assertion fails in the presence of pre-7.2.0 closed indices because such indices don't even have routing table entries.

Relates #33888
Closes #91470

This assertion fails in the presence of pre-7.2.0 closed indices because
such indices don't even have routing table entries.

Relates elastic#33888
Closes elastic#91470
@DaveCTurner DaveCTurner added >test Issues or PRs that are addressing/adding tests :Distributed/Allocation All issues relating to the decision making around placing a shard (both master logic & on the nodes) v8.6.1 v8.7.0 labels Nov 21, 2022
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Meta label for distributed team label Nov 21, 2022
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

@DaveCTurner DaveCTurner added auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) auto-backport-and-merge Automatically create backport pull requests and merge when ready labels Nov 21, 2022
@elasticsearchmachine elasticsearchmachine merged commit 1c9a39c into elastic:main Nov 21, 2022
@DaveCTurner DaveCTurner deleted the 2022-11-21-desired-balance-with-ancient-closed-indices branch November 21, 2022 15:38
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Nov 21, 2022
This assertion fails in the presence of pre-7.2.0 closed indices because
such indices don't even have routing table entries.

Relates elastic#33888 Closes elastic#91470
elasticsearchmachine pushed a commit that referenced this pull request Nov 21, 2022
This assertion fails in the presence of pre-7.2.0 closed indices because
such indices don't even have routing table entries.

Relates #33888 Closes #91470
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport-and-merge Automatically create backport pull requests and merge when ready auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Distributed/Allocation All issues relating to the decision making around placing a shard (both master logic & on the nodes) Team:Distributed Meta label for distributed team >test Issues or PRs that are addressing/adding tests v8.6.1 v8.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] :qa:full-cluster-restart:v7.0.1#upgradedClusterTest failing
3 participants