You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by jc...@apache.org on 2018/11/15 23:12:16 UTC

[geode] 01/01: GEODE-6062: Use random server port for ConfigureEvictionThroughGfsh

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

jchen21 pushed a commit to branch feature/GEODE-6062
in repository https://gitbox.apache.org/repos/asf/geode.git

commit bca143d4449cabedd7beae8b9adc2cc10bf8686f
Author: Galen O'Sullivan <go...@pivotal.io>
AuthorDate: Thu Nov 15 15:07:30 2018 -0800

    GEODE-6062: Use random server port for ConfigureEvictionThroughGfsh
    
    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 <go...@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",