[Flaky Test] Fix Flaky Test ShuffleForcedMergePolicyTests.testDiagnostics

[kkewwei]

Description

There is a situation where all segments are triggered to merged into one segment before forceMerging. When forceMerge is called, there will be no segments to merge.

reader.leaves() contains the segments being merged.

It may be caused by upgrading to lucene10.

Related Issues

Resolves #17294

Check List

• Functionality includes testing.
• API changes companion pull request created, if applicable.
• Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[github-actions]

❌ Gradle check result for 3649f79: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[github-actions]

❌ Gradle check result for 3649f79: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[github-actions]

❌ Gradle check result for 3649f79: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[github-actions]

❌ Gradle check result for 3649f79: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[github-actions]

❌ Gradle check result for 30137d5: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[kkewwei]

❌ Gradle check result for 30137d5: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

MetadataCreateIndexServiceTests.testCreateIndexWithContextDisabled #17291

[cwperks]

@kkewwei I don't think your branch is synced completely with the latest commits on main. I see that https://github.com/kkewwei/OpenSearch/tree/fix_17294 is 4 commits behind. Can you sync and push?

[kkewwei]

@kkewwei I don't think your branch is synced completely with the latest commits on main. I see that https://github.com/kkewwei/OpenSearch/tree/fix_17294 is 4 commits behind. Can you sync and push?

@cwperks My mistake, thank you very much.

[github-actions]

✅ Gradle check result for cb41d13: SUCCESS

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.46%. Comparing base (43e589a) to head (cb41d13).
Report is 5 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #17392      +/-   ##
============================================
+ Coverage     72.37%   72.46%   +0.08%     
- Complexity    65519    65530      +11     
============================================
  Files          5291     5291              
  Lines        304302   304325      +23     
  Branches      44174    44181       +7     
============================================
+ Hits         220240   220526     +286     
+ Misses        65997    65693     -304     
- Partials      18065    18106      +41     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kkewwei]

@reta @cwperks please have a look when you are free.

[reta]

@reta @cwperks please have a look when you are free.

Thanks a lot @kkewwei !