diff --git a/CHANGELOG.md b/CHANGELOG.md index 8817379..99e00ea 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -4,6 +4,13 @@ ## [1.1.1] - 2023-12-13 +- Remove the default value for `TARANTOOL_CLUSTER_COOKIE` env variable in cartridge container. + Now it works like in cartridge in order of decreasing priority as directed by the user: + `TARANTOOL_CLUSTER_COOKIE > cartridge.cfg > default_cookie` + You can set TARANTOOL_CLUSTER_COOKIE by build-arg on image building or by env arg before container starting + ([#55](https://github.com/tarantool/testcontainers-java-tarantool/issues/55)) +- **[Breaking change]** Default routerPassword has been changed from `testapp-cluster-cookie` to `secret-cluster-cookie` + ([#55](https://github.com/tarantool/testcontainers-java-tarantool/issues/55)) - Change private to protected in TarantoolCartridgeContainer - Add support for the `TARANTOOL_VERSION` environment variable to specify the version in the image name `tarantool/tarantool:-centos7 when calling the constructor without arguments diff --git a/src/main/java/org/testcontainers/containers/TarantoolCartridgeContainer.java b/src/main/java/org/testcontainers/containers/TarantoolCartridgeContainer.java index 0c64fb0..388fd8b 100644 --- a/src/main/java/org/testcontainers/containers/TarantoolCartridgeContainer.java +++ b/src/main/java/org/testcontainers/containers/TarantoolCartridgeContainer.java @@ -87,8 +87,8 @@ public class TarantoolCartridgeContainer extends GenericContainer run.sh && chmod +x run.sh +CMD ./run.sh diff --git a/src/test/java/org/testcontainers/containers/TarantoolCartridgeBootstrapFromLuaWithFixedPortsTest.java b/src/test/java/org/testcontainers/containers/TarantoolCartridgeBootstrapFromLuaWithFixedPortsTest.java index 1c3ae14..6f1fa38 100644 --- a/src/test/java/org/testcontainers/containers/TarantoolCartridgeBootstrapFromLuaWithFixedPortsTest.java +++ b/src/test/java/org/testcontainers/containers/TarantoolCartridgeBootstrapFromLuaWithFixedPortsTest.java @@ -1,6 +1,8 @@ package org.testcontainers.containers; import java.time.Duration; +import java.util.List; +import java.util.Map; import org.junit.jupiter.api.Test; import org.rnorth.ducttape.RetryCountExceededException; @@ -11,6 +13,9 @@ import org.testcontainers.junit.jupiter.Testcontainers; import static org.junit.Assert.assertThrows; import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertFalse; +import static org.junit.jupiter.api.Assertions.assertTrue; + /** * @author Alexey Kuzin @@ -38,6 +43,38 @@ public void test_StaticClusterContainer_StartsSuccessfully_ifFilesAreCopied() th CartridgeContainerTestUtils.executeProfileReplaceSmokeTest(container); } + @Test + public void testTarantoolClusterCookieDefault() throws Exception { + Map env = container.getEnvMap(); + assertFalse(env.containsKey(TarantoolCartridgeContainer.ENV_TARANTOOL_CLUSTER_COOKIE)); + List result = container.executeCommandDecoded("return true"); + assertEquals(1, result.size()); + assertTrue((boolean) result.get(0)); + } + + @Test + public void testTarantoolClusterCookieWithEnv() throws Exception { + try(TarantoolCartridgeContainer newContainer = new TarantoolCartridgeContainer( + "Dockerfile", + "cartridge", + "cartridge/instances.yml", + "cartridge/replicasets.yml") + .withEnv(TarantoolCartridgeContainer.ENV_TARANTOOL_CLUSTER_COOKIE, "secret") + .withRouterUsername("admin") + .withRouterPassword("secret") + .withStartupTimeout(Duration.ofMinutes(5)) + .withLogConsumer(new Slf4jLogConsumer( + LoggerFactory.getLogger(TarantoolCartridgeBootstrapFromYamlTest.class)))) + { + newContainer.start(); + Map env = container.getEnvMap(); + assertFalse(env.containsKey(TarantoolCartridgeContainer.ENV_TARANTOOL_CLUSTER_COOKIE)); + List result = container.executeCommandDecoded("return true"); + assertEquals(1, result.size()); + assertTrue((boolean) result.get(0)); + } + } + @Test public void test_retryingSetupTopology_shouldWork() { try (TarantoolCartridgeContainer testContainer = diff --git a/src/test/resources/cartridge/init.lua b/src/test/resources/cartridge/init.lua index 93799bc..44b68a9 100644 --- a/src/test/resources/cartridge/init.lua +++ b/src/test/resources/cartridge/init.lua @@ -41,7 +41,6 @@ local ok, err = cartridge.cfg({ 'app.roles.custom', 'migrator', }, - cluster_cookie = 'testapp-cluster-cookie', }) assert(ok, tostring(err))