From 8f0d07791997da7472102ae9ab4e70d56c9d52c9 Mon Sep 17 00:00:00 2001 From: DJ Gregor Date: Fri, 4 Nov 2022 20:37:47 -0400 Subject: [PATCH] Remove "Starting an elasticsearch container" message in constructor At this point in the constructor, we are not actually starting Elasticsearch, so this message can lead to confusion. I didn't find any of the other container classes that output a message like this in their constructor. When the container is actually started, we still get a log message from GenericContainer, anyway. --- .../org/testcontainers/elasticsearch/ElasticsearchContainer.java | 1 - 1 file changed, 1 deletion(-) diff --git a/modules/elasticsearch/src/main/java/org/testcontainers/elasticsearch/ElasticsearchContainer.java b/modules/elasticsearch/src/main/java/org/testcontainers/elasticsearch/ElasticsearchContainer.java index edef04e001e..1cb43f68bb7 100644 --- a/modules/elasticsearch/src/main/java/org/testcontainers/elasticsearch/ElasticsearchContainer.java +++ b/modules/elasticsearch/src/main/java/org/testcontainers/elasticsearch/ElasticsearchContainer.java @@ -96,7 +96,6 @@ public ElasticsearchContainer(final DockerImageName dockerImageName) { dockerImageName.assertCompatibleWith(DEFAULT_IMAGE_NAME, DEFAULT_OSS_IMAGE_NAME); this.isOss = dockerImageName.isCompatibleWith(DEFAULT_OSS_IMAGE_NAME); - logger().info("Starting an elasticsearch container using [{}]", dockerImageName); withNetworkAliases("elasticsearch-" + Base58.randomString(6)); withEnv("discovery.type", "single-node"); // Sets default memory of elasticsearch instance to 2GB