Skip to content

Remove SearchWithIndexBlocksIT (replaced by ITs in Serverless) #132013

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

Merged
merged 1 commit into from
Jul 30, 2025

Conversation

benchaplin
Copy link
Contributor

Resolves #131662, #131693, #131695, #131698.

In order to create these tests, I had to manually mutate cluster state using setState. This approach was unreliable (~1% fail rate)—sometimes other cluster state updates would occur in the middle of my update loop leading to inconsistent states.

I've given up on manually setting the block and opted for higher-level ITs in stateless where the block is added as it would be added in prod: when an index is new and search shards are not ready.

@benchaplin benchaplin added >non-issue Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch :Search Foundations/Search Catch all for Search Foundations labels Jul 28, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search-foundations (Team:Search Foundations)

@elasticsearchmachine elasticsearchmachine added v9.2.0 serverless-linked Added by automation, don't add manually labels Jul 28, 2025
Copy link
Contributor

@drempapis drempapis left a comment

Choose a reason for hiding this comment

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

LGTM

@benchaplin benchaplin merged commit 650ce32 into elastic:main Jul 30, 2025
34 checks passed
afoucret pushed a commit to afoucret/elasticsearch that referenced this pull request Jul 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>non-issue :Search Foundations/Search Catch all for Search Foundations serverless-linked Added by automation, don't add manually Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch v9.2.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] SearchWithIndexBlocksIT testSearchShardsOnIndicesWithIndexRefreshBlocks failing
3 participants