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.

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

[ci] SearchIT.testSearchWithSignificantTermsAgg #32855

Closed
andyb-elastic opened this issue Aug 14, 2018 · 2 comments
Closed

[ci] SearchIT.testSearchWithSignificantTermsAgg #32855

andyb-elastic opened this issue Aug 14, 2018 · 2 comments
Assignees
@markharwood
Labels
>test-failure Triaged test failures from CI

Comments

@andyb-elastic
Copy link
Contributor

On 6.x, reproduces locally. Looks like this test was added in #32774

https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+6.x+intake/2257/console

build-2257.txt

REPRODUCE WITH: ./gradlew :client:rest-high-level:integTestRunner -Dtests.seed=4207CF2E79C9815D -Dtests.class=org.elasticsearch.client.SearchIT -Dtests.method="testSearchWithSignificantTermsAgg" -Dtests.security.manager=true -Dtests.locale=pl -Dtests.timezone=Africa/Douala -Dcompiler.java=10 -Druntime.java=8

  1> [2018-08-14T17:59:39,804][INFO ][o.e.c.SearchIT           ] [testSearchWithSignificantTermsAgg] after test
FAILURE 0.76s | SearchIT.testSearchWithSignificantTermsAgg <<< FAILURES!
   > Throwable #1: java.lang.AssertionError: expected:<3> but was:<1>
   >    at __randomizedtesting.SeedInfo.seed([4207CF2E79C9815D:D189593C6CEFC175]:0)
   >    at org.elasticsearch.client.SearchIT.testSearchWithSignificantTermsAgg(SearchIT.java:299)
   >    at java.lang.Thread.run(Thread.java:748)

Others
https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+6.x+intake/2256/console
https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+6.x+intake/2255/console
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

andyb-elastic added a commit that referenced this issue Aug 14, 2018
@andyb-elastic
[test] mute testSearchWithSignificantTermsAgg
516fdba 
For #32855
andyb-elastic added a commit that referenced this issue Aug 14, 2018
@andyb-elastic
[test] mute testSearchWithSignificantTermsAgg
84aa87f 
For #32855
markharwood added a commit that referenced this issue Aug 15, 2018
@markharwood
Removed flaky test. Looks like randomisation makes these assertions u…
ecc6dfb 
…nreliable.

This test is superfluous - it was added to address #32770 but it later turned out there was an existing test that just required a fix to provide the missing test coverage.

Closes #32855
@markharwood
Copy link
Contributor

markharwood commented Aug 15, 2018 •
edited
Loading

Ugh. Sorry for the noise. Removed now redundant test in ecc6dfb but forgot about unused imports which are addressed in 2582b38

Also removed from master via eb812c8

markharwood added a commit that referenced this issue Aug 15, 2018
@markharwood
Removed flaky test. Looks like randomisation makes these assertions u…
eb812c8 
…nreliable.

This test is superfluous - it was added to address #32770 but it later turned out there was an existing test that just required a fix to provide the missing test coverage.

Closes #32855
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@markharwood markharwood

Labels
>test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@markharwood @elasticmachine @andyb-elastic