[zookeeper] branch branch-3.6 updated: ZOOKEEPER-3981: Flaky test MultipleAddressTest::testGetValidAddressWithNotValid

[symat]

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

symat pushed a commit to branch branch-3.6
in repository https://gitbox.apache.org/repos/asf/zookeeper.git


The following commit(s) were added to refs/heads/branch-3.6 by this push:
     new 79a91b9  ZOOKEEPER-3981: Flaky test 
MultipleAddressTest::testGetValidAddressWithNotValid
79a91b9 is described below

commit 79a91b95341bf417f3237d33846db19b39831712
Author: Michael Han <h...@apache.org>
AuthorDate: Sat Oct 24 13:18:10 2020 +0000

    ZOOKEEPER-3981: Flaky test 
MultipleAddressTest::testGetValidAddressWithNotValid
    
    Problem:
    
    Test MultipleAddressTest::testGetValidAddressWithNotValid might fail 
deterministically when the address it's using, 10.0.0.1, is reachable, as per 
https://tools.ietf.org/html/rfc5735 10.0.0.1 might be allocatable to private 
network usage. In fact, the router address of my ISP is assigned this IP, 
leading to this test always failing for me.
    
    Solution:
    
    Replace the address with 240.0.0.0, which is reserved for future use and 
less likely to be reachable.
    
    Author: Michael Han <mich...@lianghan.org>
    Author: Michael Han <h...@apache.org>
    
    Reviewers: Christopher Tubbs <ctubb...@apache.org>, Mate Szalay-Beko 
<sy...@apache.org>
    
    Closes #1511 from hanm/fixut
    
    (cherry picked from commit 3ff30a6427fdc2f13e60ff530800eca55ff02c9a)
    Signed-off-by: Mate Szalay-Beko <sy...@apache.org>
---
 .../java/org/apache/zookeeper/server/quorum/MultipleAddressesTest.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/MultipleAddressesTest.java
 
b/zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/MultipleAddressesTest.java
index 0e1615d..e060fa8 100644
--- 
a/zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/MultipleAddressesTest.java
+++ 
b/zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/MultipleAddressesTest.java
@@ -105,7 +105,8 @@ public class MultipleAddressesTest {
 
     @Test(expected = NoRouteToHostException.class)
     public void testGetValidAddressWithNotValid() throws 
NoRouteToHostException {
-        MultipleAddresses multipleAddresses = new MultipleAddresses(new 
InetSocketAddress("10.0.0.1", 22));
+        // IP chosen because it is reserved for documentation/examples and 
should be unreachable (RFC 5737)
+        MultipleAddresses multipleAddresses = new MultipleAddresses(new 
InetSocketAddress("203.0.113.1", 22));
         multipleAddresses.getReachableAddress();
     }