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

FlushIT#testDoNotRenewSyncedFlushWhenAllSealed fails on CI #32436

Closed
javanna opened this issue Jul 27, 2018 · 2 comments
Closed

FlushIT#testDoNotRenewSyncedFlushWhenAllSealed fails on CI #32436

javanna opened this issue Jul 27, 2018 · 2 comments
Assignees
@dnhatn
Labels
:Distributed Indexing/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >test-failure Triaged test failures from CI

Comments

@javanna
Copy link
Member

javanna commented Jul 27, 2018

https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+6.x+intake/2168/1

This one reproduces for me with the seed on both master and 6.x.

16:35:10 FAILURE 0.51s J3 | FlushIT.testDoNotRenewSyncedFlushWhenAllSealed <<< FAILURES!
16:35:10    > Throwable #1: java.lang.AssertionError: 
16:35:10    > Expected: <3>
16:35:10    >      but: was <0>
16:35:10    > 	at __randomizedtesting.SeedInfo.seed([43D9764F86E61665:1EE62FCD108854E7]:0)
16:35:10    > 	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
16:35:10    > 	at org.elasticsearch.indices.flush.FlushIT.testDoNotRenewSyncedFlushWhenAllSealed(FlushIT.java:316)
16:35:10    > 	at java.lang.Thread.run(Thread.java:748)

16:35:10   2> REPRODUCE WITH: ./gradlew :server:integTest -Dtests.seed=43D9764F86E61665 -Dtests.class=org.elasticsearch.indices.flush.FlushIT -Dtests.method="testDoNotRenewSyncedFlushWhenAllSealed" -Dtests.security.manager=true -Dtests.locale=ru -Dtests.timezone=Europe/Helsinki
@javanna javanna added >test-failure Triaged test failures from CI :Distributed Indexing/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. labels Jul 27, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

javanna added a commit that referenced this issue Jul 27, 2018
@javanna
[TEST] Mute failing FlushIT test
39d0890 
See #32436
javanna added a commit that referenced this issue Jul 27, 2018
@javanna
[TEST] Mute failing FlushIT test
dcb5d24 
See #32436
@dnhatn dnhatn self-assigned this Jul 27, 2018
@dnhatn
Copy link
Member

dnhatn commented Jul 27, 2018

This test started failing after we upgraded to Lucene-7.5. I will look at this soon.

@dnhatn dnhatn closed this as completed in 6e98615 Jul 28, 2018
dnhatn added a commit that referenced this issue Jul 28, 2018
@dnhatn
TEST: Avoid deletion in FlushIT
096c578 
Due to the recent change in LUCENE-8263, a merge can be triggered if the
deletion ration is higher than 33%. An in-progress merge can prevent a
synced-flush from issuing.

This commit avoids deletes by using different docIds.

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

@dnhatn dnhatn

Labels
:Distributed Indexing/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@javanna @dnhatn @elasticmachine