Browse Source

[ML] Remove distributed jobs test trace logging (#80877)

There has been trace logging in BasicDistributedJobsIT for #63980
since February. This was added in #69198, on the basis that #69136
had probably fixed a problem but if not then the trace logging
would be useful. Since the tests have not been failing since they
were reenabled this logging can now be removed.
David Roberts 3 years ago
parent
commit
501149b025

+ 0 - 35
x-pack/plugin/ml/src/internalClusterTest/java/org/elasticsearch/xpack/ml/integration/BasicDistributedJobsIT.java

@@ -45,8 +45,6 @@ import org.elasticsearch.xpack.core.ml.job.config.JobTaskState;
 import org.elasticsearch.xpack.ml.MachineLearning;
 import org.elasticsearch.xpack.ml.MachineLearning;
 import org.elasticsearch.xpack.ml.MlInitializationService;
 import org.elasticsearch.xpack.ml.MlInitializationService;
 import org.elasticsearch.xpack.ml.support.BaseMlIntegTestCase;
 import org.elasticsearch.xpack.ml.support.BaseMlIntegTestCase;
-import org.junit.After;
-import org.junit.Before;
 
 
 import java.io.IOException;
 import java.io.IOException;
 import java.util.Collection;
 import java.util.Collection;
@@ -68,39 +66,6 @@ import static org.hamcrest.Matchers.notNullValue;
 
 
 public class BasicDistributedJobsIT extends BaseMlIntegTestCase {
 public class BasicDistributedJobsIT extends BaseMlIntegTestCase {
 
 
-    @Before
-    // upping the logging due to potential failures in: https://github.com/elastic/elasticsearch/issues/63980
-    public void setLogging() {
-        client().admin()
-            .cluster()
-            .prepareUpdateSettings()
-            .setPersistentSettings(
-                Settings.builder()
-                    .put("logger.org.elasticsearch.xpack.ml.action.TransportCloseJobAction", "TRACE")
-                    .put("logger.org.elasticsearch.xpack.ml.action.TransportOpenJobAction", "TRACE")
-                    .put("logger.org.elasticsearch.xpack.ml.job.task.OpenJobPersistentTasksExecutor", "TRACE")
-                    .put("logger.org.elasticsearch.xpack.ml.job.process.autodetect.AutodetectProcessManager", "TRACE")
-                    .build()
-            )
-            .get();
-    }
-
-    @After
-    public void unsetLogging() {
-        client().admin()
-            .cluster()
-            .prepareUpdateSettings()
-            .setPersistentSettings(
-                Settings.builder()
-                    .putNull("logger.org.elasticsearch.xpack.ml.action.TransportCloseJobAction")
-                    .putNull("logger.org.elasticsearch.xpack.ml.action.TransportOpenJobAction")
-                    .putNull("logger.org.elasticsearch.xpack.ml.job.task.OpenJobPersistentTasksExecutor")
-                    .putNull("logger.org.elasticsearch.xpack.ml.job.process.autodetect.AutodetectProcessManager")
-                    .build()
-            )
-            .get();
-    }
-
     public void testFailOverBasics() throws Exception {
     public void testFailOverBasics() throws Exception {
         internalCluster().ensureAtLeastNumDataNodes(4);
         internalCluster().ensureAtLeastNumDataNodes(4);
         ensureStableCluster(4);
         ensureStableCluster(4);