Skip to content

Conversation

cwperks
Copy link
Member

@cwperks cwperks commented Oct 14, 2025

Description

This PR ensure jdk 21 minimum compatibility across all sub-projects (a few projects had it set to Java 8) and create replacement for java.security.Policy which is marked for removal

Related Issues

Related to #18339

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.

…e repacement for java.security.Policy which is marked for removal

Signed-off-by: Craig Perkins <[email protected]>
Copy link
Contributor

❌ Gradle check result for fa5211c: 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?

Signed-off-by: Craig Perkins <[email protected]>
Copy link
Contributor

❌ Gradle check result for c0d489c: 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?

Copy link
Contributor

❌ Gradle check result for 9f25041: 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?

Copy link
Contributor

✅ Gradle check result for 9f25041: SUCCESS

Copy link

codecov bot commented Oct 15, 2025

Codecov Report

❌ Patch coverage is 3.26087% with 89 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.99%. Comparing base (4ab0ae9) to head (9f25041).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
...n/java/org/opensearch/secure_sm/policy/Policy.java 0.00% 85 Missing ⚠️
...va/org/opensearch/gradle/OpenSearchJavaPlugin.java 0.00% 2 Missing ⚠️
...rg/opensearch/gradle/OpenSearchTestBasePlugin.java 0.00% 1 Missing ⚠️
...rg/opensearch/tools/cli/plugin/PluginSecurity.java 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main   #19631      +/-   ##
============================================
+ Coverage     72.96%   72.99%   +0.02%     
+ Complexity    70655    70570      -85     
============================================
  Files          5725     5726       +1     
  Lines        323697   323764      +67     
  Branches      46875    46882       +7     
============================================
+ Hits         236198   236330     +132     
+ Misses        68436    68357      -79     
- Partials      19063    19077      +14     

☔ 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.

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.

1 participant