f5-icontrol 0.2.3 → 0.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6eb8a54f54995d658cc9096ff1394345d3ce7457
-  data.tar.gz: f46d475017ab3cbfd1bd3e0441e182e9f0fce79a
+  metadata.gz: b41af60e6a4f2679ddf9c5cadedf90677f0ad54f
+  data.tar.gz: 4d8912cb7bba726e570e3ee78a172fc08c5ff74f
 SHA512:
-  metadata.gz: 6eb32f8c9832dc6cb76a1dc5445deb8ff55b8282a78d41665d7d6cd8ba23c791eaf3fa340e444a969b3acf9357b07489e24e314b6b4eb7737d4b1712429ac527
-  data.tar.gz: e7a27999ccfec01a30e1b9495ac733775d5c071f04732f2474c89c1b37a648314875bbe6bba5c46175203b671fe7b06037d2b38f51c020032822a3da3156bbaa
+  metadata.gz: d2f31200ca7c977dafbbf6b3eddcc496b6896cdcb41b28dcfeda1ca124058a0113decc58cc420a46fc3011169dbb7eb8beed4e64d247dd1d161d0ee252aeb57e
+  data.tar.gz: e39a4d2fa1a3743ea8d181bb719a8bbf430050f56750ffd67315c2051ab4bbc03ca8839c072894aa53b4fa2aff845bfe79a0d4393b9311e9e55d44bb381a8dd4

data/CHANGELOG.md

@@ -1,4 +1,11 @@
 # Changelog
+## 0.2.4
+swalberg - support setting pool member ratios
+## 0.2.3
+Fix typo from previous release
+## 0.2.2
+davidalpert - Add some enumerations to localLB
+Ignore this release there was a typo
 ## 0.2.1
 davidalpert - Add some enumerations to localLB
 

data/f5-icontrol.gemspec

@@ -21,10 +21,11 @@ Gem::Specification.new do |spec|
   spec.add_dependency "savon", "~> 2.0"
   spec.add_dependency "thor"
 
-  spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "awesome_print"
+  spec.add_development_dependency "bundler", "~> 1.3"
+  spec.add_development_dependency "byebug"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "rspec"
-  spec.add_development_dependency "webmock"
   spec.add_development_dependency "vcr"
+  spec.add_development_dependency "webmock"
 end

data/lib/f5/cli/application.rb

@@ -210,8 +210,8 @@ module F5
         response = client.Networking.VLAN.create_v2(
           vlans: { item: [ name ] },
           vlan_ids: { item: [ vid ] },
-          members: {  item: [ item: memberdata ] },
-          failsafe_states: {  item: [ 'STATE_DISABLED' ] },
+          members: { item: [ item: memberdata ] },
+          failsafe_states: { item: [ 'STATE_DISABLED' ] },
           timeouts: { item: [ 100 ] } #???
         )
         puts response
@@ -286,19 +286,19 @@ module F5
         response = client.LocalLB.Pool.set_member_session_enabled_state(
           pool_names: { item: [ pool ] },
           members: { item: [ set ] },
-          session_states: {  item: [ set.map { "STATE_ENABLED" } ] }
+          session_states: { item: [ set.map { "STATE_ENABLED" } ] }
         )
 
         response = client.LocalLB.Pool.set_member_monitor_state(
           pool_names: { item: [ pool ] },
           members: { item: [ set ] },
-          monitor_states: {  item: [ set.map { "STATE_ENABLED" } ] }
+          monitor_states: { item: [ set.map { "STATE_ENABLED" } ] }
         )
 
       end
 
       desc "disable POOL MEMBERS", "Disables the given members"
-      method_option :force, default: false, desc: "Forces the node offline (only active connections allowed)"
+      method_option :force, default: false, type: :boolean, desc: "Forces the node offline (only active connections allowed)"
       def disable(pool, *members)
         set = pool_members(pool).select do |m|
           members.include? m[:address]
@@ -307,19 +307,38 @@ module F5
         response = client.LocalLB.Pool.set_member_session_enabled_state(
           pool_names: { item: [ pool ] },
           members: { item: [ set ] },
-          session_states: {  item: [ set.map { "STATE_DISABLED" } ] }
+          session_states: { item: [ set.map { "STATE_DISABLED" } ] }
         )
 
         if options[:force]
           response = client.LocalLB.Pool.set_member_monitor_state(
             pool_names: { item: [ pool ] },
             members: { item: [ set ] },
-            monitor_states: {  item: [ set.map { "STATE_DISABLED" } ] }
+            monitor_states: { item: [ set.map { "STATE_DISABLED" } ] }
           )
         end
       end
 
+      desc "setratio --ratio RATIO POOL MEMBERS", "Sets the dynamic ratio of the given members to RATIO"
+      method_option :ratio, type: :numeric, desc: "The node's new dynamic ratio", required: true
+      def setratio(pool, *members)
+        set = address_port_list_from_pool(pool, members)
+        response = client.LocalLB.Pool.set_member_ratio(
+          pool_names: { item: [ pool ] },
+          members: { item: [ set ] },
+          ratios: { item: [ set.map { options[:ratio] } ] }
+        )
+
+      end
+
       private
+
+      def address_port_list_from_pool(pool, members)
+        pool_members(pool).select do |m|
+          members.include?(m[:address]) || members.include?(m[:address].gsub(%r{^/Common/}, ''))
+        end
+      end
+
       def pool_members(pool)
         response = client.LocalLB.Pool.get_member_v2(pool_names: { item: [ pool ] } )
 

data/lib/f5/icontrol/version.rb

@@ -1,5 +1,5 @@
 module F5
   module Icontrol
-    VERSION = "0.2.3"
+    VERSION = "0.2.4"
   end
 end

data/spec/cli/pool_spec.rb

@@ -8,26 +8,80 @@ describe F5::Cli::Pool do
 
   let(:client) { double }
 
+  context "setratio" do
+    let(:pool) { double("Pool") }
+    let(:out) { 'yup' }
+
+    before do
+      allow(pool).to receive(:get_member_v2).and_return(members)
+      allow(client).to receive_message_chain("LocalLB", "Pool") { pool }
+      subject.options = { ratio: 42 }
+    end
+
+    context "one member" do
+      let(:expected) {
+        {
+          pool_names: { item: [ 'mypool' ] },
+          members: { item: [ [ { address: '/Common/node1', port: '80' } ] ] },
+          ratios: {  item: [ [ 42 ] ] }
+        }
+      }
+      let(:members) {
+        {:item=>{:item=>{:address=>"/Common/node1", :port=>"80"}, :"@a:array_type"=>"iControl:Common.AddressPort[1]"}, :"@s:type"=>"A:Array", :"@a:array_type"=>"iControl:Common.AddressPort[][1]"}
+      }
+
+      it "calls the API to set the member" do
+        expect(pool).to receive(:set_member_ratio).with(expected).and_return(out)
+
+        subject.setratio('mypool', 'node1')
+      end
+    end
+
+    context "two members" do
+      let(:expected) {
+        {
+          pool_names: { item: [ 'mypool' ] },
+          members: { item: [ [ { address: '/Common/node1', port: '80' }, { address: '/Common/node2', port: '80' } ] ] },
+          ratios: {  item: [[ 42, 42 ]] }
+        }
+      }
+
+      let(:members) {
+        {:item=>{:item=>[{:address=>"/Common/node1", :port=>"80"}, {:address=>"/Common/node2", :port=>"80"}], :"@a:array_type"=>"iControl:Common.AddressPort[2]"}, :"@s:type"=>"A:Array", :"@a:array_type"=>"iControl:Common.AddressPort[][1]"}
+      }
+
+      it "calls the API to set the members" do
+        expect(pool).to receive(:set_member_ratio).with(expected).and_return(out)
+
+        subject.setratio('mypool', 'node1', 'node2')
+      end
+    end
+  end
+
   context "list" do
     let(:output) { capture(:stdout) { subject.list } }
 
+    before do
+      allow(client).to receive_message_chain("LocalLB", "Pool", "get_list") { api_result }
+    end
+
     context "no pools" do
+      let(:api_result) { { item: nil } }
       it "indicates there are no pools" do
-        allow(client).to receive_message_chain("LocalLB", "Pool", "get_list") { { item: nil } }
         expect(output).to include("No pools found")
       end
     end
 
     context "one pool" do
+      let(:api_result) { { item: "/Common/foobar" } }
       it "lists the pools" do
-        allow(client).to receive_message_chain("LocalLB", "Pool", "get_list") { { item: "/Common/foobar" } }
         expect(output).to eq("/Common/foobar\n")
       end
     end
 
     context "two pools" do
+      let(:api_result) { { item: [ "/Common/one", "/Common/two" ] } }
       it "lists the pools" do
-        allow(client).to receive_message_chain("LocalLB", "Pool", "get_list") { { item: [ "/Common/one", "/Common/two" ] } }
         expect(output).to eq("/Common/one\n/Common/two\n")
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: f5-icontrol
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.2.4
 platform: ruby
 authors:
 - Sean Walberg
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-29 00:00:00.000000000 Z
+date: 2017-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: savon
@@ -38,6 +38,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: awesome_print
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -53,7 +67,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
-  name: awesome_print
+  name: byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -95,7 +109,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: webmock
+  name: vcr
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -109,7 +123,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: vcr
+  name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="