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

[BUG] Test Failure org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.testHighWatermarkNotExceeded #3579

Closed
opensearch-ci-bot opened this issue Jun 14, 2022 · 7 comments · Fixed by #4012
Assignees
Labels
bug Something isn't working cicd flaky-test Random test failure that succeeds on second run

Comments

@opensearch-ci-bot
Copy link
Collaborator

@opensearch-project/opensearch-core team, please take a look at Build 5977 in gradle_check log for more information.

@opensearch-ci-bot opensearch-ci-bot added bug Something isn't working cicd labels Jun 14, 2022
@xuezhou25
Copy link
Contributor

> Task :server:internalClusterTest

REPRODUCE WITH: ./gradlew ':server:internalClusterTest' --tests "org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.testHighWatermarkNotExceeded" -Dtests.seed=2B0CB509EFB4F322 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=ar-TN -Dtests.timezone=America/Whitehorse -Druntime.java=17

org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT > testHighWatermarkNotExceeded FAILED
    java.lang.AssertionError: Mismatching shard routings: []
    Expected: a collection with size <1>
         but: collection size was <0>
        at __randomizedtesting.SeedInfo.seed([2B0CB509EFB4F322:C22D54BB6F723ACC]:0)
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
        at org.junit.Assert.assertThat(Assert.java:964)
        at org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.lambda$assertBusyWithDiskUsageRefresh$5(DiskThresholdDeciderIT.java:362)
        at org.opensearch.test.OpenSearchTestCase.assertBusy(OpenSearchTestCase.java:1049)
        at org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.assertBusyWithDiskUsageRefresh(DiskThresholdDeciderIT.java:355)
        at org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.testHighWatermarkNotExceeded(DiskThresholdDeciderIT.java:188)

@tlfeng
Copy link
Collaborator

tlfeng commented Jun 14, 2022

Also occurred in #3583 (comment)

@tlfeng tlfeng changed the title Jenkins Randomized Gradle Check Failed on main [BUG] Test Failure org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.testHighWatermarkNotExceeded Jun 14, 2022
@kotwanikunal
Copy link
Member

Also in #3568 #3595 #3576

@tlfeng
Copy link
Collaborator

tlfeng commented Jun 17, 2022

The flaky test failure is caused by the commit fc54154 / PR #3537 that upgrades Lucene version to 9.3.
The solution may be related to 055a9fc / #3591

The assertion failed at https://github.com/opensearch-project/OpenSearch/blob/2.0.1/server/src/internalClusterTest/java/org/opensearch/cluster/routing/allocation/decider/DiskThresholdDeciderIT.java#L188

@dreamer-89
Copy link
Member

Another occurrence #3646

This test has started failing more often recently.

@sachinpkale
Copy link
Member

Failing for #3703

@Rishikesh1159
Copy link
Member

Rishikesh1159 commented Jul 5, 2022

This test started failing after upgrading Lucene snapshot verison to "9.3.0-snapshot-b7231bb". Merge on refresh is enabled by default after upgrading. This test passes if Merge on refresh is not enabled by default. Thanks to @reta for already putting out a PR to allow users to enable merge on refresh by a index setting. Once that PR get's merged we can leverage that change in this integ test using index setting. That should fix this test 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 cicd flaky-test Random test failure that succeeds on second run
Projects
None yet
8 participants