Skip to content

[AUTOCUT] Gradle Check Flaky Test Report for SearchServiceTests #18238

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

Closed
opensearch-ci-bot opened this issue May 8, 2025 · 3 comments · Fixed by #18301
Closed

[AUTOCUT] Gradle Check Flaky Test Report for SearchServiceTests #18238

opensearch-ci-bot opened this issue May 8, 2025 · 3 comments · Fixed by #18301
Labels
autocut flaky-test Random test failure that succeeds on second run Search Search query, autocomplete ...etc >test-failure Test failure from CI, local build, etc.

Comments

@opensearch-ci-bot
Copy link
Collaborator

opensearch-ci-bot commented May 8, 2025

Flaky Test Report for SearchServiceTests

Noticed the SearchServiceTests has some flaky, failing tests that failed during post-merge actions.

Details

Git Reference Merged Pull Request Build Details Test Name
f2cc655 18104 57849 org.opensearch.search.SearchServiceTests.testConcurrentSegmentSearchWithRandomizedModeSettings
fb77cc7 16908 57852 org.opensearch.search.SearchServiceTests.testConcurrentSegmentSearchWithRandomizedModeSettings

The other pull requests, besides those involved in post-merge actions, that contain failing tests with the SearchServiceTests class are:

For more details on the failed tests refer to OpenSearch Gradle Check Metrics dashboard.

@opensearch-ci-bot opensearch-ci-bot added >test-failure Test failure from CI, local build, etc. untriaged flaky-test Random test failure that succeeds on second run autocut labels May 8, 2025
@andrross
Copy link
Member

andrross commented May 8, 2025

This can be reproduced pretty easily by running on repeat:

./gradlew ':server:test' --tests "org.opensearch.search.SearchServiceTests.testConcurrentSegmentSearchWithRandomizedModeSettings" -Dtests.iters=100

@Vikasht34 @bowenlan-amzn @jed326 I'm assuming this is related to enabling concurrent search by default. Can someone take a look here?

@andrross andrross added the Search Search query, autocomplete ...etc label May 14, 2025
@sandeshkr419
Copy link
Contributor

sandeshkr419 commented May 14, 2025

https://build.ci.opensearch.org/job/gradle-check/58060/consoleText

Capturing the failure error:

REPRODUCE WITH: ./gradlew ':server:test' --tests "org.opensearch.search.SearchServiceTests.testConcurrentSegmentSearchWithRandomizedModeSettings" -Dtests.seed=95D69893E135E51F -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=luy -Dtests.timezone=Etc/GMT+2 -Druntime.java=21

SearchServiceTests > testConcurrentSegmentSearchWithRandomizedModeSettings FAILED
    java.lang.AssertionError: expected:<false> but was:<true>
        at __randomizedtesting.SeedInfo.seed([95D69893E135E51F:FBDD89F429A0843D]:0)
        at org.junit.Assert.fail(Assert.java:89)
        at org.junit.Assert.failNotEquals(Assert.java:835)
        at org.junit.Assert.assertEquals(Assert.java:120)
        at org.junit.Assert.assertEquals(Assert.java:146)
        at org.opensearch.search.SearchServiceTests.testConcurrentSegmentSearchWithRandomizedModeSettings(SearchServiceTests.java:1571)

Probable related PR: #15363

@kkewwei
Copy link
Collaborator

kkewwei commented May 15, 2025

It seems be related to #17978.

Previously the default value was false, it had nothing to do with aggregation. Now default value is auto in case mode setting is not set, it is also decided by aggregationSupportsConcurrent, We should take aggregationSupportsConcurrent into account.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autocut flaky-test Random test failure that succeeds on second run Search Search query, autocomplete ...etc >test-failure Test failure from CI, local build, etc.
Projects
Status: Done
5 participants