commit | c3ea0c8dacfd3e5341facff9d051ff6a3803fac1 | [log] [tgz] |
---|---|---|
author | Tiago Palma <tiago.f.palma@gmail.com> | Mon Apr 08 10:04:04 2019 +0100 |
committer | Tiago Palma <tiago.f.palma@gmail.com> | Tue Apr 09 10:34:21 2019 +0000 |
tree | 7fb8f6dcc9216c33a1223507bdd5c6ba61eb6fe5 | |
parent | 3870bf6b18756544ef28da43df9dae7c8d81dfbe [diff] |
Remove hardcoded localhost from ZK test Zookeeper test container should not be using localhost Gerrit CI build slaves are ephemeral docker containers that could run on a different host. Change-Id: If6de3eb017691c8133a8812278c803a63789526d
diff --git a/src/test/java/com/googlesource/gerrit/plugins/multisite/validation/dfsrefdb/zookeeper/ZookeeperTestContainerSupport.java b/src/test/java/com/googlesource/gerrit/plugins/multisite/validation/dfsrefdb/zookeeper/ZookeeperTestContainerSupport.java index db8fe5a..7b9ebb5 100644 --- a/src/test/java/com/googlesource/gerrit/plugins/multisite/validation/dfsrefdb/zookeeper/ZookeeperTestContainerSupport.java +++ b/src/test/java/com/googlesource/gerrit/plugins/multisite/validation/dfsrefdb/zookeeper/ZookeeperTestContainerSupport.java
@@ -73,7 +73,7 @@ container.start(); Integer zkHostPort = container.getMappedPort(2181); Config splitBrainconfig = new Config(); - String connectString = "localhost:" + zkHostPort; + String connectString = container.getContainerIpAddress() + ":" + zkHostPort; splitBrainconfig.setBoolean("ref-database", null, "enabled", true); splitBrainconfig.setString("ref-database", "zookeeper", "connectString", connectString); splitBrainconfig.setString(