- Notifications
You must be signed in to change notification settings - Fork 25.5k
Ensure green step in synonyms rule yaml test #114400
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
Ensure green step in synonyms rule yaml test #114400
Conversation
Fixes test issue serverless 2922.
Pinging @elastic/es-search-relevance (Team:Search Relevance) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Thanks for fixing this @kingherc ! For my education - why fast refresh now requires to wait for green, in this specific case? 🤔 |
@carlosdelest with recent changes, searches (I assume including getting synonym rules) will run on search/replica shards in stateless also for fast refresh indices. So we need to wait for green == search shards allocated. Interestingly the bwc CI on elasticsearch did not pass on the yml now. Investigating the cluster logs, I see it has unassigned shards due to:
among others. I can see also
@carlosdelest might you know what's the issue in the above? I cannot seem to reproduce it locally. Was the yml test supposed to be working on a yellow cluster (meaning some shards could not be allocated)? |
@kingherc , I think that is due to some of the tests done in rest-api-spec/test/synonyms/110_synonyms_invalid.yml . Those tests create an invalid synonym chain, which would fail when recovering index shards that have an invalid synonym. That would cause the health of the cluster not to be green as the shards for the index are not recoverable. This tests is meant for versions > 8.16.0, as specified by the Could we limit the cluster_health check to the index for the corresponding test, to avoid any other interaction with other indices that may not be in a green status? |
…ss-2922-synonyms-fast-refresh
Thanks for the explanation @carlosdelest ! Pushed your recommendation to the PR, fingers crossed. |
💚 Backport successful
|
Fixes test issue serverless 2922.
Fixes test issue serverless 2922.
Fixes test issue serverless 2922.
Fixes test issue serverless 2922.