Skip to content

Fix build due to phasing off SecurityManager usage in favor of Java Agent #1404

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

pranu2502
Copy link
Contributor

Description

Fix build due to phasing off SecurityManager usage in favor of Java Agent

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

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.

@peterzhuamazon
Copy link
Member

The docker test not related to the issue.

@peterzhuamazon peterzhuamazon marked this pull request as ready for review April 11, 2025 16:37
@peterzhuamazon peterzhuamazon merged commit d58ac1e into opensearch-project:main Apr 11, 2025
16 of 17 checks passed
pranu2502 added a commit to pranu2502/index-management that referenced this pull request Apr 15, 2025
peterzhuamazon pushed a commit that referenced this pull request Apr 15, 2025
… of Java-agent. (#1406)

* Revert "Fix build due to phasing off SecurityManager usage in favor of Java Agent (#1404)"

This reverts commit d58ac1e.

Signed-off-by: Pranav Reddy <[email protected]>

* Using java-agent gradle plugin to phase off Security Manager in favor of Java-agent.

Signed-off-by: Pranav Reddy <[email protected]>

---------

Signed-off-by: Pranav Reddy <[email protected]>
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.

2 participants