Merge pull request #147 from bdunne/miq_apache_cleanup

bybusiness is the only load balancing method used

lib/gems/pending/util/miq_apache/miq_apache.rb

@@ -188,13 +188,7 @@ def self.install_default_config(opts = {})
     end
 
     def self.create_balancer_config(opts = {})
-      lbmethod = case opts[:lbmethod]
-                 when :busy then     :bybusyness
-                 when :traffic then  :bytraffic
-                 else            :byrequests
-                 end
-
-      "<Proxy balancer://#{opts[:cluster]}/ lbmethod=#{lbmethod}>\n</Proxy>\n"
+      "<Proxy balancer://#{opts[:cluster]}/ lbmethod=bybusyness>\n</Proxy>\n"
     end
 
     def self.create_redirects_config(opts = {})

spec/util/miq_apache/conf_spec.rb

@@ -10,20 +10,6 @@
   end
 
   context "building balancer config" do
-    context "lbmethod" do
-      it "should return a config with lbmethod of 'byrequests' by default" do
-        expect(MiqApache::Conf.create_balancer_config).to include("lbmethod=byrequests")
-      end
-
-      it "should return a config with lbmethod of 'bytraffic' if passed :traffic" do
-        expect(MiqApache::Conf.create_balancer_config(:lbmethod => :traffic)).to include("lbmethod=bytraffic")
-      end
-
-      it "should return a config with lbmethod of 'bybusyness' if passed :busy" do
-        expect(MiqApache::Conf.create_balancer_config(:lbmethod => :busy)).to include("lbmethod=bybusyness")
-      end
-    end
-
     it "should set cluster name" do
       expect(MiqApache::Conf.create_balancer_config(:cluster => 'evmcluster_ui')).to include("Proxy balancer://evmcluster_ui/")
     end