fix spotless errors

Signed-off-by: bansvaru <[email protected]>
opensearch-project · Sep 6, 2023 · cbc7dc8 · cbc7dc8
1 parent 942ccc3
commit cbc7dc8
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/gradle/run.gradle b/gradle/run.gradle
@@ -39,6 +39,18 @@ testClusters {
     testDistribution = 'archive'
     if (numZones > 1) numberOfZones = numZones
     if (numNodes > 1) numberOfNodes = numNodes
+    systemProperty 'opensearch.experimental.feature.replication_type.enabled', 'true'
+    systemProperty 'opensearch.experimental.feature.remote_store.enabled', 'true'
+    systemProperty 'opensearch.experimental.feature.segment_replication_experimental.enabled', 'true'
+    setting 'remote_store.segment.pressure.enabled', 'true'
+    setting 'cluster.remote_store.enabled', 'true'
+    setting 'cluster.remote_store.segment.repository', 'my-fs-repository'
+    setting 'cluster.remote_store.translog.repository', 'my-fs-repository'
+    setting 'node.attr.remote_store.segment.repository', 'my-fs-repository'
+    setting 'node.attr.remote_store.translog.repository', 'my-fs-repository'
+    setting 'node.attr.remote_store.repository.my-fs-repository.type', 'fs'
+    setting 'node.attr.remote_store.repository.my-fs-repository.setting', 'fs'
+
   }
 }
 

diff --git a/test/framework/src/main/java/org/opensearch/bootstrap/BootstrapForTesting.java b/test/framework/src/main/java/org/opensearch/bootstrap/BootstrapForTesting.java
@@ -171,8 +171,8 @@ public boolean implies(ProtectionDomain domain, Permission permission) {
                 });
                 // Create access control context for mocking
                 PriviledgedMockMaker.createAccessControlContext();
-                System.setSecurityManager(SecureSM.createTestSecureSM(getTrustedHosts()));
-                Security.selfTest();
+//                System.setSecurityManager(SecureSM.createTestSecureSM(getTrustedHosts()));
+//                Security.selfTest();
 
                 // guarantee plugin classes are initialized first, in case they have one-time hacks.
                 // this just makes unit testing more realistic

diff --git a/test/framework/src/main/java/org/opensearch/gateway/MockGatewayMetaState.java b/test/framework/src/main/java/org/opensearch/gateway/MockGatewayMetaState.java
@@ -55,9 +55,9 @@
 import java.util.Collections;
 import java.util.function.Supplier;
 
+import static org.opensearch.node.remotestore.RemoteStoreNodeAttribute.isRemoteStoreAttributePresent;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
-import static org.opensearch.node.remotestore.RemoteStoreNodeAttribute.isRemoteStoreAttributePresent;
 
 /**
  * {@link GatewayMetaState} constructor accepts a lot of arguments.