commit 7c62e1232cf8063d3b0f0dc59978da69710602b5
author karanmehta93 <karanmehta93@gmail.com> Mon Apr 29 19:28:35 2019 -0700
committer Sijie Guo <guosijie@gmail.com> Tue Apr 30 10:28:35 2019 +0800
tree c3487e2a4ba7149f684a4c25f005d1de482b6d6a
parent 42e77802c025964d826c3b57e8f18c4bc464c481

ISSUE #2073: ReadOnlyBookieTest#testBookieContinueWritingIfMulti…

…pleLedgersPresent is flaky

Descriptions of the changes in this PR:

Updated the test to use random port number for bookie startup instead of predefined one.

### Motivation

ReadOnlyBookieTest#testBookieContinueWritingIfMultipleLedgersPresent test is flaky due to this reason.

### Changes

(Describe: what changes you have made)

Master Issue: #2073 




Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Sijie Guo <sijie@apache.org>

This closes #2074 from karanmehta93/master, closes #2073

bookkeeper-server/src/test/java/org/apache/bookkeeper/test/ReadOnlyBookieTest.java

diff --git a/bookkeeper-server/src/test/java/org/apache/bookkeeper/test/ReadOnlyBookieTest.java b/bookkeeper-server/src/test/java/org/apache/bookkeeper/test/ReadOnlyBookieTest.java
index 35b2fd3..329779c 100644
--- a/bookkeeper-server/src/test/java/org/apache/bookkeeper/test/ReadOnlyBookieTest.java
+++ b/bookkeeper-server/src/test/java/org/apache/bookkeeper/test/ReadOnlyBookieTest.java
@@ -245,7 +245,7 @@
         }
 
         ServerConfiguration newConf = newServerConfiguration(
-                conf.getBookiePort() + 1,
+                PortManager.nextFreePort(),
                 ledgerDirs[0], ledgerDirs);
         bsConfs.add(newConf);
         bs.add(startBookie(newConf));