You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2019/06/06 03:55:27 UTC

[hbase] branch master updated: HBASE-22546 TestRegionServerHostname#testRegionServerHostname fails reliably for me

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

stack pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/master by this push:
     new a826f41  HBASE-22546 TestRegionServerHostname#testRegionServerHostname fails reliably for me
a826f41 is described below

commit a826f41e8307cd4349669b8ecc42eec73e79ccce
Author: stack <st...@apache.org>
AuthorDate: Wed Jun 5 12:53:20 2019 -0700

    HBASE-22546 TestRegionServerHostname#testRegionServerHostname fails reliably for me
---
 .../apache/hadoop/hbase/regionserver/TestRegionServerHostname.java   | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerHostname.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerHostname.java
index a90b40a..70d4202 100644
--- a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerHostname.java
+++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionServerHostname.java
@@ -96,11 +96,12 @@ public class TestRegionServerHostname {
       // iterate through host addresses and use each as hostname
       while (addrList.hasMoreElements()) {
         InetAddress addr = addrList.nextElement();
-        if (addr.isLoopbackAddress() || addr.isLinkLocalAddress() || addr.isMulticastAddress()) {
+        if (addr.isLoopbackAddress() || addr.isLinkLocalAddress() || addr.isMulticastAddress() ||
+            !addr.isSiteLocalAddress()) {
           continue;
         }
         String hostName = addr.getHostName();
-        LOG.info("Found " + hostName + " on " + ni);
+        LOG.info("Found " + hostName + " on " + ni + ", addr=" + addr);
 
         TEST_UTIL.getConfiguration().set(HRegionServer.MASTER_HOSTNAME_KEY, hostName);
         TEST_UTIL.getConfiguration().set(HRegionServer.RS_HOSTNAME_KEY, hostName);