albertogpz commented on a change in pull request #7465:
URL: https://github.com/apache/geode/pull/7465#discussion_r831306956
##########
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:
I suggest we get rid of the .gfsh file here and inline the command and
the same for the next two gfsh invocations.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]