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

[CI] flaky test failure - o.o.indices.stats.IndexStatsIT.testFilterCacheStats #2501

Closed
tlfeng opened this issue Mar 18, 2022 · 5 comments · Fixed by #5963
Closed

[CI] flaky test failure - o.o.indices.stats.IndexStatsIT.testFilterCacheStats #2501

tlfeng opened this issue Mar 18, 2022 · 5 comments · Fixed by #5963
Assignees
Labels
bug Something isn't working flaky-test Random test failure that succeeds on second run

Comments

@tlfeng
Copy link
Collaborator

tlfeng commented Mar 18, 2022

Caught from PR #2424 (comment):

❌ Gradle Check failure 90f143a
Log 3504

Reports 3504

In log 3504:

> Task :server:internalClusterTest

REPRODUCE WITH: ./gradlew ':server:internalClusterTest' --tests "org.opensearch.indices.stats.IndexStatsIT.testFilterCacheStats" -Dtests.seed=D48688166EC0CB86 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=fr -Dtests.timezone=GMT -Druntime.java=17

org.opensearch.indices.stats.IndexStatsIT > testFilterCacheStats FAILED
    java.lang.AssertionError: expected:<0> but was:<1>
        at __randomizedtesting.SeedInfo.seed([D48688166EC0CB86:B1B97553DEC8CCFF]:0)
        at org.junit.Assert.fail(Assert.java:89)
        at org.junit.Assert.failNotEquals(Assert.java:835)
        at org.junit.Assert.assertEquals(Assert.java:647)
        at org.junit.Assert.assertEquals(Assert.java:633)
        at org.opensearch.indices.stats.IndexStatsIT.testFilterCacheStats(IndexStatsIT.java:1225)
@tlfeng tlfeng added the flaky-test Random test failure that succeeds on second run label Mar 18, 2022
@tlfeng tlfeng added the bug Something isn't working label Mar 18, 2022
@tlfeng tlfeng changed the title [CI] o.o.indices.stats.IndexStatsIT.testFilterCacheStats [CI] flaky test failure - o.o.indices.stats.IndexStatsIT.testFilterCacheStats Mar 18, 2022
@tlfeng tlfeng added the v2.0.0 Version 2.0.0 label Mar 18, 2022
@anasalkouz anasalkouz removed the v2.0.0 Version 2.0.0 label Apr 12, 2022
@andrross
Copy link
Member

andrross commented Nov 4, 2022

@dblock
Copy link
Member

dblock commented Nov 8, 2022

#3633

@andrross
Copy link
Member

#5365

@Rishikesh1159
Copy link
Member

#5332

@mch2
Copy link
Member

mch2 commented Jan 20, 2023

I've gotten this to reproduce locally. Its failing on this line that asserts there is nothing in the query cache. At this point in the test no searches have been initiated from the test but there is occasionally an entry in the cache.

Up until that point in the test the only reqs made are to index a random set of docs. Looks like a query is being made somewhere from inside engine on the index path that reuses the same querycache, I think that is here where only if assertions are enabled we check if the doc is not actually in the index from indexIntoLucene. I don't think this is related to randomization, but that the query is only occasionally cached or evicted before the test fetches stats.

I've nulled out / used DisabledQueryCache impl on this searcher and am no longer able to repro the failure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working flaky-test Random test failure that succeeds on second run
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants