Skip to content

Commit

Permalink
[ML] Remove distributed jobs test trace logging (elastic#80877)
Browse files Browse the repository at this point in the history
There has been trace logging in BasicDistributedJobsIT for elastic#63980
since February. This was added in elastic#69198, on the basis that elastic#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.
  • Loading branch information
droberts195 committed Nov 19, 2021
1 parent 9c1c999 commit b498ef4
Showing 1 changed file with 0 additions and 35 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,6 @@
import org.elasticsearch.xpack.core.ml.job.config.JobTaskState;
import org.elasticsearch.xpack.ml.MachineLearning;
import org.elasticsearch.xpack.ml.support.BaseMlIntegTestCase;
import org.junit.After;
import org.junit.Before;

import java.io.IOException;
import java.util.Collection;
Expand All @@ -66,39 +64,6 @@

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 {
internalCluster().ensureAtLeastNumDataNodes(4);
ensureStableCluster(4);
Expand Down

0 comments on commit b498ef4

Please sign in to comment.