[GitHub] [geode] upthewaterspout commented on a change in pull request #7465: GEODE-10127: Use correct hostname in WAN with LB test

[GitBox <gi...@apache.org>]

upthewaterspout commented on a change in pull request #7465:
URL: https://github.com/apache/geode/pull/7465#discussion_r832435925



##########
File path: geode-assembly/src/acceptanceTest/java/org/apache/geode/cache/wan/SeveralGatewayReceiversWithSamePortAndHostnameForSendersTest.java
##########
@@ -101,8 +100,8 @@
   @BeforeClass
   public static void beforeClass() throws Exception {
     // Start locator
-    docker.execForService("locator", "gfsh", "run",
-        "--file=/geode/scripts/geode-starter-locator.gfsh");
+    docker.execForService("locator", "gfsh", "-e",
+        startLocatorCommand());
     // Start server1
     docker.execForService("server1", "gfsh", "run",
         "--file=/geode/scripts/geode-starter-server1.gfsh");

Review comment:
       Good idea! I turned all of the scripts into `gfsh -e` commands inline in the java code. Thanks!




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org