Skip to content

[Flaky Test] Fix flaky test in SecureReactorNetty4HttpServerTransportTests with reproducible seed#19327

Merged
andrross merged 4 commits intoopensearch-project:mainfrom
cwperks:fix-tz-issue
Sep 18, 2025
Merged

[Flaky Test] Fix flaky test in SecureReactorNetty4HttpServerTransportTests with reproducible seed#19327
andrross merged 4 commits intoopensearch-project:mainfrom
cwperks:fix-tz-issue

Conversation

@cwperks
Copy link
Copy Markdown
Member

@cwperks cwperks commented Sep 17, 2025

Description

This PR is an attempt to solve more flakey tests in SecureReactorNetty4HttpServerTransportTests

This PR includes a solution for the random seed:

./gradlew ':plugins:transport-reactor-netty4:test' --tests 'org.opensearch.http.reactor.netty4.ssl.SecureReactorNetty4HttpServerTransportTests.testLargeCompressedResponse' -Dtests.seed=7EFC5F02D7E19F20 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=bgc-IN -Dtests.timezone=Pacific/Marquesas -Druntime.java=24

Related Issues

Related to #17486

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.

…ortTests

Signed-off-by: Craig Perkins <cwperx@amazon.com>
@cwperks cwperks requested a review from a team as a code owner September 17, 2025 20:11
Signed-off-by: Craig Perkins <cwperx@amazon.com>
@github-actions
Copy link
Copy Markdown
Contributor

❌ Gradle check result for fd5bc29: null

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?

Signed-off-by: Craig Perkins <cwperx@amazon.com>
@cwperks cwperks changed the title [Flakey Test] Fix flakey tests in SecureReactorNetty4HttpServerTransportTests [Flaky Test] Fix flaky tests in SecureReactorNetty4HttpServerTransportTests Sep 18, 2025
Signed-off-by: Craig Perkins <cwperx@amazon.com>
@cwperks cwperks changed the title [Flaky Test] Fix flaky tests in SecureReactorNetty4HttpServerTransportTests [Flaky Test] Fix flaky test in SecureReactorNetty4HttpServerTransportTests with reproducible seed Sep 18, 2025
@github-actions
Copy link
Copy Markdown
Contributor

✅ Gradle check result for 11ef71d: SUCCESS

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.77%. Comparing base (6c4f3d8) to head (11ef71d).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #19327      +/-   ##
============================================
- Coverage     72.88%   72.77%   -0.11%     
+ Complexity    69841    69749      -92     
============================================
  Files          5673     5673              
  Lines        320756   320756              
  Branches      46370    46370              
============================================
- Hits         233796   233445     -351     
- Misses        68102    68431     +329     
- Partials      18858    18880      +22     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@andrross andrross merged commit 1f9eb87 into opensearch-project:main Sep 18, 2025
32 checks passed
jainankitk pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 22, 2025
…producible seed (opensearch-project#19327)

Signed-off-by: Craig Perkins <cwperx@amazon.com>
jainankitk pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 22, 2025
…producible seed (opensearch-project#19327)

Signed-off-by: Craig Perkins <cwperx@amazon.com>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
jainankitk pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 22, 2025
…producible seed (opensearch-project#19327)

Signed-off-by: Craig Perkins <cwperx@amazon.com>
Signed-off-by: Ankit Jain <jainankitk@apache.org>
asimmahmood1 pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 23, 2025
…producible seed (opensearch-project#19327)

Signed-off-by: Craig Perkins <cwperx@amazon.com>
pranikum pushed a commit to pranikum/OpenSearch that referenced this pull request Sep 23, 2025
…producible seed (opensearch-project#19327)

Signed-off-by: Craig Perkins <cwperx@amazon.com>
vinaykpud pushed a commit to vinaykpud/OpenSearch that referenced this pull request Sep 26, 2025
…producible seed (opensearch-project#19327)

Signed-off-by: Craig Perkins <cwperx@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants