From f3b4f177d3bde5875938331b56e962a05a3c2007 Mon Sep 17 00:00:00 2001 From: Ryan Ernst Date: Tue, 6 May 2025 15:15:46 -0700 Subject: [PATCH] Disable SecurityManager in tests SecurityManager is already removed in 9.1 and 8.19, but those changes are too large to backport to 8.18 and 9.0. This commit changes tests there to not run with security manager to avoid security manager specific failures even though release versions only runs with entitlements. closes #127784 closes #127785 closes #127786 --- .../java/org/elasticsearch/bootstrap/BootstrapForTesting.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/framework/src/main/java/org/elasticsearch/bootstrap/BootstrapForTesting.java b/test/framework/src/main/java/org/elasticsearch/bootstrap/BootstrapForTesting.java index 0ef16e7e9f555..d426cf91fe484 100644 --- a/test/framework/src/main/java/org/elasticsearch/bootstrap/BootstrapForTesting.java +++ b/test/framework/src/main/java/org/elasticsearch/bootstrap/BootstrapForTesting.java @@ -120,7 +120,7 @@ public class BootstrapForTesting { IfConfig.logIfNecessary(); // install security manager if available and requested - if (RuntimeVersionFeature.isSecurityManagerAvailable() && systemPropertyAsBoolean("tests.security.manager", true)) { + if (RuntimeVersionFeature.isSecurityManagerAvailable() && systemPropertyAsBoolean("tests.security.manager", false)) { try { // initialize paths the same exact way as bootstrap Permissions perms = new Permissions();