From 464a03ec87b8502442b8f52d8b9254ced8cbcff4 Mon Sep 17 00:00:00 2001 From: Ryan Ernst Date: Tue, 6 May 2025 16:17:07 -0700 Subject: [PATCH] Disable SecurityManager in tests (#127789) 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();