honeymoose/OpenSearch
1
0
Fork 0
mirror of https://github.com/honeymoose/OpenSearch.git synced 2025-02-19 19:35:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

[ML] Resolve NetworkDisruptionIT (#43441)

Browse Source 
After the network disruption a partition is created,
one side of which can form a cluster the other can't.
Ensure requests are sent to a node on the correct side
of the cluster
This commit is contained in:
David Kyle 2019-06-21 10:14:58 +01:00
parent d1280339a8
commit 73221d2265
2 changed files with 1 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
x-pack/plugin/ml/src/test/java/org/elasticsearch/xpack/ml/integration/NetworkDisruptionIT.java
View File

@ -38,12 +38,6 @@ public class NetworkDisruptionIT extends BaseMlIntegTestCase {
return plugins; return plugins;
} }
// Remove this once the AwaitsFix below has been resolved
public void testDummy() {
assertTrue(true);
}
@AwaitsFix( bugUrl = "https://github.com/elastic/elasticsearch/issues/39858")
public void testJobRelocation() throws Exception { public void testJobRelocation() throws Exception {
internalCluster().ensureAtLeastNumDataNodes(5); internalCluster().ensureAtLeastNumDataNodes(5);
ensureStableCluster(5); ensureStableCluster(5);

2
x-pack/plugin/ml/src/test/java/org/elasticsearch/xpack/ml/support/BaseMlIntegTestCase.java
View File

@ -353,7 +353,7 @@ public abstract class BaseMlIntegTestCase extends ESIntegTestCase {
protected String awaitJobOpenedAndAssigned(String jobId, String queryNode) throws Exception { protected String awaitJobOpenedAndAssigned(String jobId, String queryNode) throws Exception {
PersistentTasksClusterService persistentTasksClusterService = PersistentTasksClusterService persistentTasksClusterService =
internalCluster().getInstance(PersistentTasksClusterService.class, internalCluster().getMasterName()); internalCluster().getInstance(PersistentTasksClusterService.class, internalCluster().getMasterName(queryNode));
// Speed up rechecks to a rate that is quicker than what settings would allow. // Speed up rechecks to a rate that is quicker than what settings would allow.
// The check would work eventually without doing this, but the assertBusy() below // The check would work eventually without doing this, but the assertBusy() below
// would need to wait 30 seconds, which would make the test run very slowly. // would need to wait 30 seconds, which would make the test run very slowly.