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

[7.16] Preventing unnecessary ILM policy deletions that drastically slow down SmokeTestMultiNodeClientYamlTestSuiteIT (#79946) #80053

Merged
merged 3 commits into from
Oct 29, 2021

Conversation

masseyke
Copy link
Member

Backports the following commits to 7.16:

…n SmokeTestMultiNodeClientYamlTestSuiteIT (elastic#79946)

In elastic#76791 several new default ILM policies were added. EsRestTestCase deletes all ILM policies that it does
not know about one-at-a-time. Each of these deletions causes a cluster state change that needs to be propagated
to all nodes. In a large test on a multi-node cluster (like SmokeTestMultiNodeClientYamlTestSuiteIT) this eats up a
significant amount of time -- about 30% of the runtime of the test. This was causing
SmokeTestMultiNodeClientYamlTestSuiteIT to fail with timeouts. This commit adds the new standard ILM policies to
the list of known policies not to delete.
Closes elastic#77025
Relates elastic#76791
@masseyke
Copy link
Member Author

@elasticsearchmachine update branch

@masseyke
Copy link
Member Author

@elasticmachine run elasticsearch-ci/bwc

@elasticsearchmachine elasticsearchmachine merged commit b61894a into elastic:7.16 Oct 29, 2021
@masseyke masseyke deleted the backport/7.16/pr-79946 branch October 29, 2021 16:37
@danhermann danhermann added v7.16.0 and removed v7.16.1 labels Nov 3, 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.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants