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

Add integration test for shard availability health indicator #100490

Merged
merged 4 commits into from Oct 26, 2023

Conversation

idegtiarenko
Copy link
Contributor

This change adds integration test for shard availability health indicator with initial scenarios that verify health level is not degrading during number of operations.

Related to: #99951

This change adds integration test for shard availability health indicator with
initial scenarios that verify health level is not degrading during number of
operations.
@elasticsearchmachine elasticsearchmachine added Team:Data Management Meta label for data/management team v8.12.0 and removed Team:Distributed Meta label for distributed team labels Oct 9, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

Copy link
Contributor

@gmarouli gmarouli left a comment

Choose a reason for hiding this comment

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

LGTM, just to double check, other operations like shrink or reindex will be captured by testIsGreenDuringIndexCreate, right?

@idegtiarenko
Copy link
Contributor Author

LGTM, just to double check, other operations like shrink or reindex will be captured by testIsGreenDuringIndexCreate, right?

Shrink/Split/Clone should behave similarly, I covered the last one already.
It seems both clone and reindex behave the same as regular index creation but we could always add more test cases if needed.

@idegtiarenko idegtiarenko merged commit 18b1bf5 into elastic:main Oct 26, 2023
12 checks passed
@idegtiarenko idegtiarenko deleted the shard_availability_it branch October 26, 2023 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants