albertogpz commented on a change in pull request #7330: URL: https://github.com/apache/geode/pull/7330#discussion_r811346516
########## File path: geode-wan/src/distributedTest/java/org/apache/geode/cache/wan/internal/cli/commands/WanCopyRegionCommandDUnitTest.java ########## @@ -1460,11 +1460,15 @@ public void createServersAndRegions(int locatorPort, List<VM> servers, } private void waitForWanCopyRegionCommandToStart(boolean useParallel, boolean usePartitionedRegion, - List<VM> servers) { + List<VM> servers) throws InterruptedException { + // Wait for the command execution to be registered in the service final int executionsExpected = useParallel && usePartitionedRegion ? servers.size() : 1; await().untilAsserted( () -> assertThat(getNumberOfCurrentExecutionsInServers(servers)) .isEqualTo(executionsExpected)); + // Wait some extra milliseconds to allow for the command to actually start to + // avoid flakyness in the tests. + Thread.sleep(100); Review comment: @kohlmu-pivotal Thanks for the improvements suggested. Are you planning on taking any of them? I am saying in order to tackle whichever you are not taking or all if you take none. -- 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-unsubscr...@geode.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org