rbeapi 0.5.1 → 1.0

data/lib/rbeapi/version.rb

@@ -33,5 +33,5 @@
 # #
 # Rbeapi toplevel namespace.
 module Rbeapi
-  VERSION = '0.5.1'
+  VERSION = '1.0'
 end

data/rbeapi.gemspec

@@ -31,4 +31,6 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rspec', '~> 3.0.0'
   spec.add_development_dependency 'rspec-mocks', '~> 3.0.0'
   spec.add_development_dependency 'simplecov'
+
+  spec.required_ruby_version = '>= 1.9.3'
 end

data/rbeapi.spec.tmpl

@@ -31,6 +31,18 @@ The library is freely provided to the open source community for building robust
 applications using Arista EOS eAPI. Support is provided as best effort through
 Github iusses.
 
+%package chef
+Summary:	Arista eAPI Ruby Library for Chef clients
+Group:		Development/Languages
+Requires: chef
+Requires: chef-rubygem(net_http_unix)
+Requires: chef-rubygem(inifile)
+Requires: chef-rubygem(netaddr)
+Provides: chef-rubygem(%{gem_name}) = %{version}
+Provides: chef-rubygem-%{gem_name} = %{version}
+%description chef
+The Ruby eAPI Client for Chef clients
+
 %package puppet3
 Summary:	Arista eAPI Ruby Library Puppet Enterprise 3.x agents
 Group:		Development/Languages
@@ -66,6 +78,9 @@ install %{SOURCE0} %{buildroot}/
 %files
 /%{gem_name}-%{version}.gem
 
+%files chef
+/%{gem_name}-%{version}.gem
+
 %files puppet3
 /%{gem_name}-%{version}.gem
 
@@ -73,15 +88,24 @@ install %{SOURCE0} %{buildroot}/
 /%{gem_name}-%{version}.gem
 
 %post
-GEM_OPTS="--no-document --local"
+GEM_OPTS="--no-rdoc --no-ri --local"
 gem install ${GEM_OPTS} /%{gem_name}-%{version}.gem > /dev/null 2>&1
 
 %preun
 gem uninstall %{gem_name} --version '= %{version}' > /dev/null 2>&1
 
+%post chef
+GEM=/opt/chef/embedded/bin/gem
+GEM_OPTS="--no-document --local"
+${GEM} install ${GEM_OPTS} /%{gem_name}-%{version}.gem > /dev/null 2>&1
+
+%preun chef
+GEM=/opt/chef/embedded/bin/gem
+${GEM} uninstall %{gem_name} --version '= %{version}' > /dev/null 2>&1
+
 %post puppet3
 GEM=/opt/puppet/bin/gem
-GEM_OPTS="--no-document --local"
+GEM_OPTS="--no-rdoc --no-ri --local"
 ${GEM} install ${GEM_OPTS} /%{gem_name}-%{version}.gem > /dev/null 2>&1
 
 %preun puppet3
@@ -90,7 +114,7 @@ ${GEM} uninstall %{gem_name} --version '= %{version}' > /dev/null 2>&1
 
 %post puppet-aio
 GEM=/opt/puppetlabs/puppet/bin/gem
-GEM_OPTS="--no-document --local"
+GEM_OPTS="--no-rdoc --no-ri --local"
 ${GEM} install ${GEM_OPTS} /%{gem_name}-%{version}.gem > /dev/null 2>&1
 
 %preun puppet-aio
@@ -98,6 +122,9 @@ GEM=/opt/puppetlabs/puppet/bin/gem
 ${GEM} uninstall %{gem_name} --version '= %{version}' > /dev/null 2>&1
 
 %changelog
+* Tue May 24 2016 Jere Julian - 0.6.0-1
+- Add Chef sub-package
+
 * Fri Oct 30 2015 Jere Julian - 0.4.0-1
 - Detect the location of the puppet-agent's gem install
 

data/spec/fixtures/.gitignore

@@ -0,0 +1 @@
+dut.conf

data/spec/support/matchers/switch_config_sections.rb

@@ -0,0 +1,80 @@
+#
+# Copyright (c) 2016, Arista Networks, Inc.
+# All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions are
+# met:
+#
+#   Redistributions of source code must retain the above copyright notice,
+#   this list of conditions and the following disclaimer.
+#
+#   Redistributions in binary form must reproduce the above copyright
+#   notice, this list of conditions and the following disclaimer in the
+#   documentation and/or other materials provided with the distribution.
+#
+#   Neither the name of Arista Networks nor the names of its
+#   contributors may be used to endorse or promote products derived from
+#   this software without specific prior written permission.
+#
+# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL ARISTA NETWORKS
+# BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+# CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+# SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR
+# BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
+# WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE
+# OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN
+# IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+#
+
+def get_child(section, line)
+  section.children.each do |child|
+    return child if child.line == line
+  end
+  nil
+end
+
+def section_compare(section1, section2)
+  # Verify lines are the same
+  return false unless section1.line == section2.line
+
+  # Verify cmds are the same
+  c1 = Set.new(section1.cmds)
+  c2 = Set.new(section2.cmds)
+  return false unless c1 == c2
+
+  # Return false if the number of children are different
+  return false unless section1.children.length == section2.children.length
+
+  # Using a depth first search to recursively descend through the
+  # children doing a comparison.
+  section1.children.each do |s1_child|
+    s2_child = get_child(section2, s1_child.line)
+    return false unless s2_child
+    return false unless section_compare(s1_child, s2_child)
+  end
+  true
+end
+
+RSpec::Matchers.define :section_equal do |expected|
+  if expected.class != Rbeapi::SwitchConfig::Section
+    fail 'expected is not a Rbeapi::SwitchConfig::Section class'
+  end
+
+  match do |actual|
+    if actual.class != Rbeapi::SwitchConfig::Section
+      fail 'actual is not a Rbeapi::SwitchConfig::Section class'
+    end
+
+    section_compare(actual, expected)
+  end
+
+  description do
+    'Verifies contents of section with a deep class comparison.'
+  end
+
+  diffable
+end

data/spec/system/rbeapi/api/interfaces_base_spec.rb

@@ -20,7 +20,8 @@ describe Rbeapi::Api::Interfaces do
   describe '#get' do
     context 'with interface Loopback' do
       let(:entity) do
-        { name: 'Loopback0', type: 'generic', description: '', shutdown: false }
+        { name: 'Loopback0', type: 'generic', description: '', shutdown: false,
+          load_interval: '' }
       end
 
       before { node.config(['no interface Loopback0', 'interface Loopback0']) }
@@ -33,7 +34,8 @@ describe Rbeapi::Api::Interfaces do
     context 'with interface Port-Channel' do
       let(:entity) do
         { name: 'Port-Channel1', type: 'portchannel', description: '',
-          shutdown: false, members: [], lacp_mode: 'on', minimum_links: '0',
+          shutdown: false, load_interval: '', members: [], lacp_mode: 'on',
+          minimum_links: '0',
           lacp_fallback: 'disabled', lacp_timeout: '90' }
       end
 
@@ -50,7 +52,8 @@ describe Rbeapi::Api::Interfaces do
     context 'with interface Vxlan' do
       let(:entity) do
         { name: 'Vxlan1', type: 'vxlan', description: '',
-          shutdown: false, source_interface: '', multicast_group: '',
+          shutdown: false, load_interval: '', source_interface: '',
+          multicast_group: '',
           udp_port: 4789, flood_list: [], vlans: {} }
       end
 
@@ -131,4 +134,30 @@ describe Rbeapi::Api::Interfaces do
       expect(subject.get('Loopback0')[:shutdown]).to be_falsy
     end
   end
+
+  describe '#set_load_interval' do
+    before do
+      node.config(['interface Loopback0', 'default load-interval'])
+    end
+
+    it 'sets the load-interval value on the interface' do
+      expect(subject.get('Loopback0')[:load_interval]).to eq('')
+      expect(subject.set_load_interval('Loopback0', value: '10')).to be_truthy
+      expect(subject.get('Loopback0')[:load_interval]).to eq('10')
+    end
+
+    it 'negates the load-interval' do
+      expect(subject.set_load_interval('Loopback0', value: '20')).to be_truthy
+      expect(subject.get('Loopback0')[:load_interval]).to eq('20')
+      expect(subject.set_load_interval('Loopback0', enable: false)).to be_truthy
+      expect(subject.get('Loopback0')[:load_interval]).to eq('')
+    end
+
+    it 'defaults the load-interval' do
+      expect(subject.set_load_interval('Loopback0', value: '10')).to be_truthy
+      expect(subject.get('Loopback0')[:load_interval]).to eq('10')
+      expect(subject.set_load_interval('Loopback0', default: true)).to be_truthy
+      expect(subject.get('Loopback0')[:load_interval]).to eq('')
+    end
+  end
 end

data/spec/system/rbeapi/api/interfaces_ethernet_spec.rb

@@ -15,8 +15,9 @@ describe Rbeapi::Api::Interfaces do
   describe '#get' do
     let(:entity) do
       { name: 'Ethernet1', type: 'ethernet', description: '', shutdown: false,
-        speed: 'auto', forced: false, sflow: true, flowcontrol_send: 'off',
-        flowcontrol_receive: 'off' }
+        load_interval: '', speed: 'default', sflow: true,
+        flowcontrol_send: 'off', flowcontrol_receive: 'off',
+        lacp_priority: '32768' }
     end
 
     before { node.config(['default interface Ethernet1']) }
@@ -89,13 +90,8 @@ describe Rbeapi::Api::Interfaces do
   describe '#set_speed' do
     before { node.config(['default interface Ethernet1']) }
 
-    it 'sets default true' do
-      expect(subject.set_speed('Ethernet1', default: true)).to be_truthy
-    end
-
     it 'sets enable true' do
-      expect(subject.set_speed('Ethernet1', default: false,
-                                            enable: true)).to be_falsy
+      expect(subject.set_speed('Ethernet1', enable: true)).to be_falsy
     end
   end
 
@@ -150,4 +146,56 @@ describe Rbeapi::Api::Interfaces do
       expect(subject.get('Ethernet1')[:flowcontrol_receive]).to eq('off')
     end
   end
+
+  describe '#set_load_interval' do
+    before do
+      node.config(['interface Ethernet1', 'default load-interval'])
+    end
+
+    it 'sets the load-interval value on the interface' do
+      expect(subject.get('Ethernet1')[:load_interval]).to eq('')
+      expect(subject.set_load_interval('Ethernet1', value: '10')).to be_truthy
+      expect(subject.get('Ethernet1')[:load_interval]).to eq('10')
+    end
+
+    it 'negates the load-interval' do
+      expect(subject.set_load_interval('Ethernet1', value: '20')).to be_truthy
+      expect(subject.get('Ethernet1')[:load_interval]).to eq('20')
+      expect(subject.set_load_interval('Ethernet1', enable: false)).to be_truthy
+      expect(subject.get('Ethernet1')[:load_interval]).to eq('')
+    end
+
+    it 'defaults the load-interval' do
+      expect(subject.set_load_interval('Ethernet1', value: '10')).to be_truthy
+      expect(subject.get('Ethernet1')[:load_interval]).to eq('10')
+      expect(subject.set_load_interval('Ethernet1', default: true)).to be_truthy
+      expect(subject.get('Ethernet1')[:load_interval]).to eq('')
+    end
+  end
+
+  describe '#set_lacp_priority' do
+    before do
+      node.config(['interface Ethernet1', 'default lacp port-priority'])
+    end
+
+    it 'sets the lacp port-priority value on the interface' do
+      expect(subject.get('Ethernet1')[:lacp_priority]).to eq('32768')
+      expect(subject.set_lacp_priority('Ethernet1', value: '0')).to be_truthy
+      expect(subject.get('Ethernet1')[:lacp_priority]).to eq('0')
+    end
+
+    it 'negates the lacp port-priority' do
+      expect(subject.set_lacp_priority('Ethernet1', value: '1')).to be_truthy
+      expect(subject.get('Ethernet1')[:lacp_priority]).to eq('1')
+      expect(subject.set_lacp_priority('Ethernet1', enable: false)).to be_truthy
+      expect(subject.get('Ethernet1')[:lacp_priority]).to eq('32768')
+    end
+
+    it 'defaults the lacp port-priority' do
+      expect(subject.set_lacp_priority('Ethernet1', value: '2')).to be_truthy
+      expect(subject.get('Ethernet1')[:lacp_priority]).to eq('2')
+      expect(subject.set_lacp_priority('Ethernet1', default: true)).to be_truthy
+      expect(subject.get('Ethernet1')[:lacp_priority]).to eq('32768')
+    end
+  end
 end

data/spec/system/rbeapi/api/interfaces_portchannel_spec.rb

@@ -14,7 +14,8 @@ describe Rbeapi::Api::Interfaces do
   describe '#get' do
     let(:entity) do
       { name: 'Port-Channel1', type: 'portchannel', description: '',
-        shutdown: false, members: [], lacp_mode: 'on', minimum_links: '0',
+        shutdown: false, load_interval: '', members: [], lacp_mode: 'on',
+        minimum_links: '0',
         lacp_timeout: '90', lacp_fallback: 'disabled' }
     end
 
@@ -270,4 +271,35 @@ describe Rbeapi::Api::Interfaces do
       expect(subject.get('Port-Channel1')[:lacp_timeout]).to eq('100')
     end
   end
+
+  describe '#set_load_interval' do
+    before do
+      node.config(['interface Port-Channel1', 'default load-interval'])
+    end
+
+    it 'sets the load-interval value on the interface' do
+      expect(subject.get('Port-Channel1')[:load_interval]).to eq('')
+      expect(subject.set_load_interval('Port-Channel1',
+                                       value: '10')).to be_truthy
+      expect(subject.get('Port-Channel1')[:load_interval]).to eq('10')
+    end
+
+    it 'negates the load-interval' do
+      expect(subject.set_load_interval('Port-Channel1',
+                                       value: '20')).to be_truthy
+      expect(subject.get('Port-Channel1')[:load_interval]).to eq('20')
+      expect(subject.set_load_interval('Port-Channel1',
+                                       enable: false)).to be_truthy
+      expect(subject.get('Port-Channel1')[:load_interval]).to eq('')
+    end
+
+    it 'defaults the load-interval' do
+      expect(subject.set_load_interval('Port-Channel1',
+                                       value: '10')).to be_truthy
+      expect(subject.get('Port-Channel1')[:load_interval]).to eq('10')
+      expect(subject.set_load_interval('Port-Channel1',
+                                       default: true)).to be_truthy
+      expect(subject.get('Port-Channel1')[:load_interval]).to eq('')
+    end
+  end
 end

data/spec/system/rbeapi/api/interfaces_vxlan_spec.rb

@@ -14,6 +14,7 @@ describe Rbeapi::Api::Interfaces do
   describe '#get' do
     let(:entity) do
       { name: 'Vxlan1', type: 'vxlan', description: '', shutdown: false,
+        load_interval: '',
         source_interface: '', multicast_group: '', udp_port: 4789,
         flood_list: [], vlans: {} }
     end
@@ -91,6 +92,32 @@ describe Rbeapi::Api::Interfaces do
     end
   end
 
+  describe '#set_load_interval' do
+    before do
+      node.config(['interface Vxlan1', 'default load-interval'])
+    end
+
+    it 'sets the load-interval value on the interface' do
+      expect(subject.get('Vxlan1')[:load_interval]).to eq('')
+      expect(subject.set_load_interval('Vxlan1', value: '10')).to be_truthy
+      expect(subject.get('Vxlan1')[:load_interval]).to eq('10')
+    end
+
+    it 'negates the load-interval' do
+      expect(subject.set_load_interval('Vxlan1', value: '20')).to be_truthy
+      expect(subject.get('Vxlan1')[:load_interval]).to eq('20')
+      expect(subject.set_load_interval('Vxlan1', enable: false)).to be_truthy
+      expect(subject.get('Vxlan1')[:load_interval]).to eq('')
+    end
+
+    it 'defaults the load-interval' do
+      expect(subject.set_load_interval('Vxlan1', value: '10')).to be_truthy
+      expect(subject.get('Vxlan1')[:load_interval]).to eq('10')
+      expect(subject.set_load_interval('Vxlan1', default: true)).to be_truthy
+      expect(subject.get('Vxlan1')[:load_interval]).to eq('')
+    end
+  end
+
   describe '#set_source_interface' do
     before { node.config(['no interface Vxlan1', 'interface Vxlan1']) }
 

data/spec/system/rbeapi/api/ipinterfaces_spec.rb

@@ -13,7 +13,8 @@ describe Rbeapi::Api::Ipinterfaces do
 
   describe '#get' do
     let(:entity) do
-      { address: '77.99.99.99/24', mtu: '1500', helper_addresses: [] }
+      { address: '77.99.99.99/24', mtu: '1500', helper_addresses: [],
+        load_interval: '' }
     end
 
     before do
@@ -150,5 +151,37 @@ describe Rbeapi::Api::Ipinterfaces do
         .to be_truthy
       expect(subject.get('Ethernet1')[:helper_addresses].sort).to be_empty
     end
+
+    it 'raises an ArgumentError if opts value is not an array' do
+      expect { subject.set_helper_addresses('Ethernet1', value: '123') }
+        .to raise_error(ArgumentError)
+    end
+  end
+
+  describe '#set_load_interval' do
+    before do
+      node.config(['default interface Ethernet1', 'interface Ethernet1',
+                   'no switchport'])
+    end
+
+    it 'sets the load-interval value on the interface' do
+      expect(subject.get('Ethernet1')[:load_interval]).to eq('')
+      expect(subject.set_load_interval('Ethernet1', value: '10')).to be_truthy
+      expect(subject.get('Ethernet1')[:load_interval]).to eq('10')
+    end
+
+    it 'negates the load-interval' do
+      expect(subject.set_load_interval('Ethernet1', value: '20')).to be_truthy
+      expect(subject.get('Ethernet1')[:load_interval]).to eq('20')
+      expect(subject.set_load_interval('Ethernet1', enable: false)).to be_truthy
+      expect(subject.get('Ethernet1')[:load_interval]).to eq('')
+    end
+
+    it 'defaults the load-interval' do
+      expect(subject.set_load_interval('Ethernet1', value: '10')).to be_truthy
+      expect(subject.get('Ethernet1')[:load_interval]).to eq('10')
+      expect(subject.set_load_interval('Ethernet1', default: true)).to be_truthy
+      expect(subject.get('Ethernet1')[:load_interval]).to eq('')
+    end
   end
 end