OpenSearch icon indicating copy to clipboard operation
OpenSearch copied to clipboard

Fips smoke test

Open beanuwave opened this issue 7 months ago • 10 comments

Description

Create project to run smoke tests to verify server can handle some basic functionality under FIPS JVM.

Related Issues

Resolves RFC

Check List

  • [x] 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.

beanuwave avatar May 27 '25 17:05 beanuwave

:x: Gradle check result for b3cded0160ac973a5daf44711b8b36afea359821: 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[bot] avatar May 27 '25 17:05 github-actions[bot]

:x: Gradle check result for 320cb5370abffac0a59feab8f99779cf9e6f1262: 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[bot] avatar May 28 '25 08:05 github-actions[bot]

:x: Gradle check result for 059654721575fea6aacd345d62ed4b63d80141a4: 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[bot] avatar May 28 '25 08:05 github-actions[bot]

fips_build_tooling needs to be merged before this can be opened for review.

beanuwave avatar May 28 '25 10:05 beanuwave

:white_check_mark: Gradle check result for 3689020d646b6b18d9000b9bb990bae60132e027: SUCCESS

github-actions[bot] avatar May 28 '25 10:05 github-actions[bot]

Codecov Report

Attention: Patch coverage is 0% with 11 lines in your changes missing coverage. Please review.

Project coverage is 72.72%. Comparing base (3078649) to head (575c850). Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
...a/org/opensearch/RandomnessProviderNameAction.java 0.00% 9 Missing :warning:
.../main/java/org/opensearch/FipsSmokeTestPlugin.java 0.00% 2 Missing :warning:
Additional details and impacted files
@@             Coverage Diff              @@
##               main   #18376      +/-   ##
============================================
+ Coverage     72.64%   72.72%   +0.08%     
- Complexity    68070    68120      +50     
============================================
  Files          5537     5539       +2     
  Lines        313404   313415      +11     
  Branches      45476    45476              
============================================
+ Hits         227658   227928     +270     
+ Misses        67241    66889     -352     
- Partials      18505    18598      +93     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

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

codecov[bot] avatar May 28 '25 10:05 codecov[bot]

:white_check_mark: Gradle check result for 71af700646673d644b2ec1eb656037d402636a79: SUCCESS

github-actions[bot] avatar Jun 02 '25 13:06 github-actions[bot]

@beanuwave is there anything remaining before this is taken out of draft?

cwperks avatar Jun 06 '25 15:06 cwperks

:white_check_mark: Gradle check result for 575c85083ae17731845f30fcc81622bffab2d9e3: SUCCESS

github-actions[bot] avatar Jun 13 '25 14:06 github-actions[bot]

Note: The codecov report is inaccurate for this PR. The new test module does cover these lines in Randomness.java, but since Randomness is in the server/ module its not getting picked up by the new javaRestTest included in this PR.

cwperks avatar Jun 13 '25 16:06 cwperks