Fix "externalCluster" LBFactorySimple setting in LBFactoryMW
authorAaron Schulz <aschulz@wikimedia.org>
Mon, 19 Sep 2016 22:13:43 +0000 (15:13 -0700)
committerAaron Schulz <aschulz@wikimedia.org>
Mon, 19 Sep 2016 22:13:43 +0000 (15:13 -0700)
Fixes regression from d175b391ae2a

Change-Id: I4bbc875e2fcd021e342959a37475ec356cb58cce

includes/db/loadbalancer/LBFactoryMW.php
includes/libs/rdbms/lbfactory/LBFactorySimple.php

index e943a8a..f8647a3 100644 (file)
@@ -94,8 +94,8 @@ abstract class LBFactoryMW {
                                }
                                $lbConf['servers'] = [ $server ];
                        }
-                       if ( !isset( $lbConf['externalServers'] ) ) {
-                               $lbConf['externalServers'] = $mainConfig->get( 'ExternalServers' );
+                       if ( !isset( $lbConf['externalClusters'] ) ) {
+                               $lbConf['externalClusters'] = $mainConfig->get( 'ExternalServers' );
                        }
                } elseif ( $lbConf['class'] === 'LBFactoryMulti' ) {
                        if ( isset( $lbConf['serverTemplate'] ) ) {
index 0476cf2..0949092 100644 (file)
@@ -87,7 +87,7 @@ class LBFactorySimple extends LBFactory {
         */
        protected function newExternalLB( $cluster, $domain = false ) {
                if ( !isset( $this->externalClusters[$cluster] ) ) {
-                       throw new InvalidArgumentException( __METHOD__ . ": Unknown cluster \"$cluster\"" );
+                       throw new InvalidArgumentException( __METHOD__ . ": Unknown cluster \"$cluster\"." );
                }
 
                return $this->newLoadBalancer( $this->externalClusters[$cluster] );