Skip to content

[CI] TransportSearchActionTests.testShouldPreFilterSearchShards fails on CI #54156

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.

Sign up for GitHub

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

Jump to bottom
Closed
tlrx opened this issue Mar 25, 2020 · 3 comments
Closed

[CI] TransportSearchActionTests.testShouldPreFilterSearchShards fails on CI #54156

tlrx opened this issue Mar 25, 2020 · 3 comments
Assignees
@jimczi
Labels
:Search/Search Search-related issues that do not fall into other categories >test-failure Triaged test failures from CI

Comments

@tlrx
Copy link
Member

tlrx commented Mar 25, 2020

The test failed several times since it was added in #53873:

It reproduces with:

./gradlew ':server:test' --tests "org.elasticsearch.action.search.TransportSearchActionTests.testShouldPreFilterSearchShardsWithReadOnly" -Dtests.seed=B4C0C67AB112D4B5 -Dtests.security.manager=true -Dtests.locale=ar-SA -Dtests.timezone=Australia/West -Dcompiler.java=13

Since it was added very recently I'm not going to mute it (I think that it will be fixed soon).
@tlrx tlrx added :Search/Search Search-related issues that do not fall into other categories >test-failure Triaged test failures from CI labels Mar 25, 2020
@elasticmachine
Copy link
Collaborator

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

@tlrx
Copy link
Member Author

tlrx commented Mar 25, 2020

@jimczi it failed 3 times yesterday on master so if you don't have the time to look at it I can mute it straightforward.

@jimczi
Copy link
Contributor

jimczi commented Mar 25, 2020

Thanks @tlrx , I am on it but let's mute since it fails relatively often.

tlrx added a commit to tlrx/elasticsearch that referenced this issue Mar 25, 2020
@tlrx
Mute TransportSearchActionTests.testShouldPreFilterSearchShards
830a5e3 
Relates elastic#53873
Relates elastic#54156
This was referenced Mar 25, 2020
Merged
Merged
tlrx added a commit that referenced this issue Mar 25, 2020
@tlrx
Mute TransportSearchActionTests.testShouldPreFilterSearchShards (#54157)
ecb071c 
Relates #53873
Relates #54156
tlrx added a commit that referenced this issue Mar 25, 2020
@tlrx
Mute TransportSearchActionTests.testShouldPreFilterSearchShards (#54158)
b6e4822 
Relates #53873
Relates #54156
@jimczi jimczi closed this as completed in 769aae0 Mar 25, 2020
jimczi added a commit that referenced this issue Mar 25, 2020
@jimczi
Fix off-by one error in TransportSearchActionTests
e380a5a 
Closes #54156
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jimczi jimczi

Labels
:Search/Search Search-related issues that do not fall into other categories >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tlrx @elasticmachine @jimczi