Skip to content

Conversation

@cwperks
Copy link
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.

@cwperks cwperks requested a review from a team as a code owner September 17, 2025 20:11
Signed-off-by: Craig Perkins <[email protected]>
@github-actions
Copy link
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 <[email protected]>
@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 <[email protected]>
@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
Contributor

✅ Gradle check result for 11ef71d: SUCCESS

@codecov
Copy link

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
jainankitk pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 22, 2025
jainankitk pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 22, 2025
asimmahmood1 pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 23, 2025
pranikum pushed a commit to pranikum/OpenSearch that referenced this pull request Sep 23, 2025
vinaykpud pushed a commit to vinaykpud/OpenSearch that referenced this pull request Sep 26, 2025
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