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

Search with failing nodes test #46060

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

henningandersen
Copy link
Contributor

Add a test case that verifies that searches running while nodes are
failing behave correctly, in particular with
allow_partial_search_results=false.

Fixed MockSearchService concurrency, assertNoInFlightContext could
have false negative result (rarely).

Split out from #45739

Add a test case that verifies that searches running while nodes are
failing behave correctly, in particular with
allow_partial_search_results=false.

Fixed MockSearchService concurrency, assertNoInFlightContext could
have false negative result (rarely).

Split out from elastic#45739
@henningandersen henningandersen added >test Issues or PRs that are addressing/adding tests :Search/Search Search-related issues that do not fall into other categories v8.0.0 v7.4.0 labels Aug 28, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

@henningandersen
Copy link
Contributor Author

ci/1 failed due to new issue, reported and fixed here #46064
@elasticmachine run elasticsearch-ci/1

@colings86 colings86 added v7.5.0 and removed v7.4.0 labels Aug 30, 2019
henningandersen added a commit that referenced this pull request Oct 2, 2019
Fixed MockSearchService concurrency, assertNoInFlightContext could
have false negative result (rarely).

Split out from #46060
Closes #47048
henningandersen added a commit that referenced this pull request Oct 2, 2019
Fixed MockSearchService concurrency, assertNoInFlightContext could
have false negative result (rarely).

Split out from #46060
Closes #47048
henningandersen added a commit that referenced this pull request Oct 2, 2019
Fixed MockSearchService concurrency, assertNoInFlightContext could
have false negative result (rarely).

Split out from #46060
Closes #47048
@jimczi jimczi added v7.6.0 and removed v7.5.0 labels Nov 12, 2019
@polyfractal polyfractal added v7.7.0 and removed v7.6.0 labels Jan 15, 2020
@henningandersen
Copy link
Contributor Author

@elasticmachine update branch

@bpintea bpintea added v7.8.0 and removed v7.7.0 labels Mar 25, 2020
@rjernst rjernst added the Team:Search Meta label for search team label May 4, 2020
@henningandersen
Copy link
Contributor Author

@elasticmachine update branch

@mark-vieira mark-vieira added v8.2.0 and removed v8.1.0 labels Feb 2, 2022
@elasticsearchmachine elasticsearchmachine changed the base branch from master to main July 22, 2022 23:14
@mark-vieira mark-vieira added v8.5.0 and removed v8.4.0 labels Jul 27, 2022
@elasticsearchmachine
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Search/Search Search-related issues that do not fall into other categories Team:Search Meta label for search team >test Issues or PRs that are addressing/adding tests v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet