This is an automated email from the ASF dual-hosted git repository.

jchen21 pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new 8e5397e  GEODE-6062: Use random server port for 
ConfigureEvictionThroughGfsh (#2862)
8e5397e is described below

commit 8e5397edbcdd6f089d7267304b3f8b5fbbe2ac7c
Author: Jianxia Chen <jc...@pivotal.io>
AuthorDate: Mon Nov 26 12:10:12 2018 -0800

    GEODE-6062: Use random server port for ConfigureEvictionThroughGfsh (#2862)
    
    This should help reduce the frequency of random CI failures for this
    test. It's still a bit worrisome that servers were using the same port
    in the first place, but that's not really what we're testing here.
    
    Co-authored-by: Galen O'Sullivan <gosulli...@pivotal.io>
    Co-authored-by: Jianxia Chen <jc...@pivotal.io>
---
 .../internal/cli/commands/ConfigureEvictionThroughGfsh.java        | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git 
a/geode-assembly/src/acceptanceTest/java/org/apache/geode/management/internal/cli/commands/ConfigureEvictionThroughGfsh.java
 
b/geode-assembly/src/acceptanceTest/java/org/apache/geode/management/internal/cli/commands/ConfigureEvictionThroughGfsh.java
index e40ec3a..125d327 100644
--- 
a/geode-assembly/src/acceptanceTest/java/org/apache/geode/management/internal/cli/commands/ConfigureEvictionThroughGfsh.java
+++ 
b/geode-assembly/src/acceptanceTest/java/org/apache/geode/management/internal/cli/commands/ConfigureEvictionThroughGfsh.java
@@ -38,7 +38,7 @@ public class ConfigureEvictionThroughGfsh {
   public void configureEvictionByEntryCount() throws Exception {
 
     GfshExecution execution = GfshScript
-        .of("start locator --name=locator", "start server --name=server",
+        .of("start locator --name=locator", "start server --name=server 
--server-port=0",
             "create region --name=region1 --eviction-action=local-destroy 
--eviction-entry-count=1000 --type=REPLICATE",
             "create region --name=region2 --eviction-action=overflow-to-disk 
--eviction-entry-count=1000 --type=REPLICATE",
             "create region --name=region3 --eviction-action=overflow-to-disk 
--eviction-entry-count=1000 --type=REPLICATE_PERSISTENT",
@@ -94,7 +94,7 @@ public class ConfigureEvictionThroughGfsh {
   @Test
   public void configureEvictionByMaxMemory() throws Exception {
     GfshExecution execution = GfshScript
-        .of("start locator --name=locator", "start server --name=server",
+        .of("start locator --name=locator", "start server --name=server 
--server-port=0",
             "create region --name=region1 --eviction-action=local-destroy 
--eviction-max-memory=1000 --type=REPLICATE",
             "create region --name=region2 --eviction-action=overflow-to-disk 
--eviction-max-memory=1000 --type=REPLICATE",
             "create region --name=region3 --eviction-action=overflow-to-disk 
--eviction-max-memory=1000 --type=REPLICATE_PERSISTENT",
@@ -160,7 +160,8 @@ public class ConfigureEvictionThroughGfsh {
   @Test
   public void configureEvictionByObjectSizer() throws Exception {
     GfshExecution execution = GfshScript
-        .of("start locator --name=locator", "start server --name=server", 
"sleep --time=1",
+        .of("start locator --name=locator", "start server --name=server 
--server-port=0",
+            "sleep --time=1",
             "deploy --jar=" + createJar().getAbsolutePath(),
             "create region --name=region1 --eviction-action=local-destroy 
--eviction-max-memory=1000 --eviction-object-sizer=MySizer --type=REPLICATE",
             "create region --name=region2 --eviction-action=overflow-to-disk 
--eviction-max-memory=1000 --eviction-object-sizer=MySizer --type=REPLICATE",

Reply via email to