Revert "HBASE-20224 Web UI is broken in standalone mode"

Broke shell tests.

This reverts commit dd9fe813ec.

hbase-client/src/test/resources/hbase-site.xml

@@ -29,11 +29,4 @@
     <name>hbase.hconnection.threads.keepalivetime</name>
     <value>3</value>
   </property>
-  <property>
-    <name>hbase.localcluster.assign.random.ports</name>
-    <value>true</value>
-    <description>
-      Assign random ports to master and RS info server (UI).
-    </description>
-  </property>
 </configuration>

hbase-mapreduce/src/test/resources/hbase-site.xml

@@ -158,11 +158,4 @@
     <name>hbase.hconnection.threads.keepalivetime</name>
     <value>3</value>
   </property>
-  <property>
-    <name>hbase.localcluster.assign.random.ports</name>
-    <value>true</value>
-    <description>
-      Assign random ports to master and RS info server (UI).
-    </description>
-  </property>
 </configuration>

hbase-procedure/src/test/resources/hbase-site.xml

@@ -41,11 +41,4 @@
       WARNING: Doing so may expose you to additional risk of data loss!
     </description>
   </property>
-  <property>
-    <name>hbase.localcluster.assign.random.ports</name>
-    <value>true</value>
-    <description>
-      Assign random ports to master and RS info server (UI).
-    </description>
-  </property>
 </configuration>

hbase-rest/src/test/resources/hbase-site.xml

@@ -139,11 +139,4 @@
     <description>Skip sanity checks in tests
     </description>
   </property>
-  <property>
-    <name>hbase.localcluster.assign.random.ports</name>
-    <value>true</value>
-    <description>
-      Assign random ports to master and RS info server (UI).
-    </description>
-  </property>
 </configuration>

hbase-server/src/main/java/org/apache/hadoop/hbase/LocalHBaseCluster.java

@@ -141,7 +141,7 @@ public class LocalHBaseCluster {
 
     // Always have masters and regionservers come up on port '0' so we don't
     // clash over default ports.
-    if (conf.getBoolean(ASSIGN_RANDOM_PORTS, false)) {
+    if (conf.getBoolean(ASSIGN_RANDOM_PORTS, true)) {
       conf.set(HConstants.MASTER_PORT, "0");
       conf.set(HConstants.REGIONSERVER_PORT, "0");
       if (conf.getInt(HConstants.REGIONSERVER_INFO_PORT, 0) != -1) {

hbase-server/src/test/java/org/apache/hadoop/hbase/MiniHBaseCluster.java

@@ -108,7 +108,7 @@ public class MiniHBaseCluster extends HBaseCluster {
          Class<? extends MiniHBaseCluster.MiniHBaseClusterRegionServer> regionserverClass)
       throws IOException, InterruptedException {
     super(conf);
-    if (conf.getBoolean(LocalHBaseCluster.ASSIGN_RANDOM_PORTS, false)) {
+    if (conf.getBoolean(LocalHBaseCluster.ASSIGN_RANDOM_PORTS, true)) {
       conf.set(HConstants.MASTER_PORT, "0");
       if (conf.getInt(HConstants.MASTER_INFO_PORT, 0) != -1) {
         conf.set(HConstants.MASTER_INFO_PORT, "0");

hbase-server/src/test/resources/hbase-site.xml

@@ -158,11 +158,4 @@
     <name>hbase.hconnection.threads.keepalivetime</name>
     <value>3</value>
   </property>
-  <property>
-    <name>hbase.localcluster.assign.random.ports</name>
-    <value>true</value>
-    <description>
-      Assign random ports to master and RS info server (UI).
-    </description>
-  </property>
 </configuration>

hbase-thrift/src/test/resources/hbase-site.xml

@@ -154,11 +154,4 @@
       Enable replay sanity checks on procedure tests.
     </description>
   </property>
-  <property>
-    <name>hbase.localcluster.assign.random.ports</name>
-    <value>true</value>
-    <description>
-      Assign random ports to master and RS info server (UI).
-    </description>
-  </property>
 </configuration>