Skip to content

Commit a8961d2

Browse files
committed
Fix failing tests with elasticsearch 1.4.3
(cherry picked from commit 040e96f) (cherry picked from commit 13c83ce)
1 parent 27546ec commit a8961d2

File tree

3 files changed

+7
-12
lines changed

3 files changed

+7
-12
lines changed

src/test/java/org/elasticsearch/discovery/azure/AbstractAzureComputeServiceTest.java

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -60,13 +60,8 @@ protected Settings settingsBuilder() {
6060
.put("cloud.azure.keystore", "dummy")
6161
.put("cloud.azure.password", "dummy")
6262
.put("cloud.azure.service_name", "dummy")
63-
.put("cloud.azure.refresh_interval", "5s")
6463
// We need the network to make the mock working
65-
.put("node.mode", "network")
66-
// Make the tests run faster
67-
.put("discovery.zen.join.timeout", "100ms")
68-
.put("discovery.zen.ping.timeout", "10ms")
69-
.put("discovery.initial_state_timeout", "300ms");
64+
.put("node.mode", "network");
7065

7166
return builder.build();
7267
}

src/test/java/org/elasticsearch/discovery/azure/AzureMinimumMasterNodesTest.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,12 @@ public AzureMinimumMasterNodesTest() {
4747
@Override
4848
protected final Settings settingsBuilder() {
4949
ImmutableSettings.Builder builder = ImmutableSettings.settingsBuilder()
50+
.put(super.settingsBuilder())
5051
.put("discovery.zen.minimum_master_nodes", 2)
51-
.put(super.settingsBuilder());
52+
// Make the test run faster
53+
.put("discovery.zen.join.timeout", "50ms")
54+
.put("discovery.zen.ping.timeout", "10ms")
55+
.put("discovery.initial_state_timeout", "100ms");
5256
return builder.build();
5357
}
5458

src/test/java/org/elasticsearch/repositories/azure/AzureSnapshotRestoreITest.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,6 @@
3333
import org.elasticsearch.cluster.ClusterState;
3434
import org.elasticsearch.common.Strings;
3535
import org.elasticsearch.common.base.Predicate;
36-
import org.elasticsearch.common.logging.ESLogger;
37-
import org.elasticsearch.common.logging.ESLoggerFactory;
3836
import org.elasticsearch.common.settings.ImmutableSettings;
3937
import org.elasticsearch.common.settings.Settings;
4038
import org.elasticsearch.common.settings.SettingsFilter;
@@ -64,8 +62,6 @@
6462
transportClientRatio = 0.0)
6563
public class AzureSnapshotRestoreITest extends AbstractAzureTest {
6664

67-
private static ESLogger logger = ESLoggerFactory.getLogger(AzureSnapshotRestoreITest.class.getName());
68-
6965
private String getRepositoryPath() {
7066
String testName = "it-".concat(Strings.toUnderscoreCase(getTestName()).replaceAll("_", "-"));
7167
return testName.contains(" ") ? Strings.split(testName, " ")[0] : testName;
@@ -370,6 +366,7 @@ public boolean apply(Object obj) {
370366
).get();
371367
client().admin().cluster().prepareDeleteRepository("test-repo").get();
372368
try {
369+
logger.info("--> remove container [{}]", container);
373370
cleanRepositoryFiles(container);
374371
} catch (StorageException | URISyntaxException e) {
375372
// We can ignore that as we just try to clean after the test
@@ -478,7 +475,6 @@ public static void cleanRepositoryFiles(String... containers) throws StorageExce
478475

479476
AzureStorageService client = new AzureStorageServiceImpl(settings, settingsFilter);
480477
for (String container : containers) {
481-
logger.info("--> remove container [{}]", container);
482478
client.removeContainer(container);
483479
}
484480
}

0 commit comments

Comments
 (0)