cisco_node_utils 1.0.1 → 1.1.0

data/lib/cisco_node_utils/interface.rb

@@ -1,6 +1,3 @@
-#
-# NXAPI implementation of Interface class
-#
 # November 2015, Chris Van Heuveln
 #
 # Copyright (c) 2015 Cisco and/or its affiliates.
@@ -17,33 +14,33 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require File.join(File.dirname(__FILE__), 'node')
+require_relative 'node_util'
 
+# Add some interface-specific constants to the Cisco namespace
 module Cisco
   IF_SWITCHPORT_MODE = {
-    :disabled   => "",
-    :access     => "access",
-    :trunk      => "trunk",
-    :fex_fabric => "fex-fabric",
-    :tunnel     => "dot1q-tunnel",
+    disabled:   '',
+    access:     'access',
+    trunk:      'trunk',
+    fex_fabric: 'fex-fabric',
+    tunnel:     'dot1q-tunnel',
   }
 
-  class Interface
+  # Interface - node utility class for general interface config management
+  class Interface < NodeUtil
     attr_reader :name
 
-    @@node = Cisco::Node.instance
-
     def initialize(name, instantiate=true)
-      raise TypeError unless name.is_a?(String)
-      raise ArgumentError unless name.length > 0
+      fail TypeError unless name.is_a?(String)
+      fail ArgumentError unless name.length > 0
       @name = name.downcase
 
       create if instantiate
     end
 
-    def Interface.interfaces
+    def self.interfaces
       hash = {}
-      intf_list = @@node.config_get("interface", "all_interfaces")
+      intf_list = config_get('interface', 'all_interfaces')
       return hash if intf_list.nil?
 
       intf_list.each do |id|
@@ -55,11 +52,11 @@ module Cisco
 
     def create
       feature_vlan_set(true) if @name[/vlan/i]
-      @@node.config_set("interface", "create", @name)
+      config_set('interface', 'create', @name)
     end
 
     def destroy
-      @@node.config_set("interface", "destroy", @name)
+      config_set('interface', 'destroy', @name)
     end
 
     ########################################################
@@ -67,61 +64,65 @@ module Cisco
     ########################################################
 
     def access_vlan
-      vlan = @@node.config_get("interface", "access_vlan", @name)
+      vlan = config_get('interface', 'access_vlan', @name)
       return default_access_vlan if vlan.nil?
       vlan.shift.to_i
     end
 
     def access_vlan=(vlan)
-      @@node.config_set("interface", "access_vlan", @name, vlan)
+      config_set('interface', 'access_vlan', @name, vlan)
     rescue Cisco::CliError => e
       raise "[#{@name}] '#{e.command}' : #{e.clierror}"
     end
 
     def default_access_vlan
-      @@node.config_get_default("interface", "access_vlan")
+      config_get_default('interface', 'access_vlan')
     end
 
     def description
-      desc = @@node.config_get("interface", "description", @name)
-      return "" if desc.nil?
+      desc = config_get('interface', 'description', @name)
+      return '' if desc.nil?
       desc.shift.strip
     end
 
     def description=(desc)
-      raise TypeError unless desc.is_a?(String)
-      desc.empty? ?
-        @@node.config_set("interface", "description", @name, "no", "") :
-        @@node.config_set("interface", "description", @name, "", desc)
+      fail TypeError unless desc.is_a?(String)
+      if desc.strip.empty?
+        config_set('interface', 'description', @name, 'no', '')
+      else
+        config_set('interface', 'description', @name, '', desc)
+      end
     rescue Cisco::CliError => e
       raise "[#{@name}] '#{e.command}' : #{e.clierror}"
     end
 
     def default_description
-      @@node.config_get_default("interface", "description")
+      config_get_default('interface', 'description')
     end
 
     def encapsulation_dot1q
-      val = @@node.config_get("interface", "encapsulation_dot1q", @name)
+      val = config_get('interface', 'encapsulation_dot1q', @name)
       return default_encapsulation_dot1q if val.nil?
       val.shift.strip.to_i
     end
 
     def encapsulation_dot1q=(val)
-      val.nil? ?
-        @@node.config_set("interface", "encapsulation_dot1q", @name, "no", "") :
-        @@node.config_set("interface", "encapsulation_dot1q", @name, "", val)
+      if val.to_s.empty?
+        config_set('interface', 'encapsulation_dot1q', @name, 'no', '')
+      else
+        config_set('interface', 'encapsulation_dot1q', @name, '', val)
+      end
     rescue Cisco::CliError => e
       raise "[#{@name}] '#{e.command}' : #{e.clierror}"
     end
 
     def default_encapsulation_dot1q
-      @@node.config_get_default("interface", "encapsulation_dot1q")
+      config_get_default('interface', 'encapsulation_dot1q')
     end
 
     def fex_feature
-      fex = @@node.config_get("fex", "feature")
-      raise "fex_feature not found" if fex.nil?
+      fex = config_get('fex', 'feature')
+      fail 'fex_feature not found' if fex.nil?
       fex.shift.to_sym
     end
 
@@ -131,32 +132,32 @@ module Cisco
 
       case fex_set
       when :enabled
-        @@node.config_set("fex", "feature_install", "") if curr == :uninstalled
-        @@node.config_set("fex", "feature", "")
+        config_set('fex', 'feature_install', '') if curr == :uninstalled
+        config_set('fex', 'feature', '')
       when :disabled
-        @@node.config_set("fex", "feature", "no") if curr == :enabled
+        config_set('fex', 'feature', 'no') if curr == :enabled
         return
       when :installed
-        @@node.config_set("fex", "feature_install", "") if curr == :uninstalled
+        config_set('fex', 'feature_install', '') if curr == :uninstalled
       when :uninstalled
-        @@node.config_set("fex", "feature", "no") if curr == :enabled
-        @@node.config_set("fex", "feature_install", "no")
+        config_set('fex', 'feature', 'no') if curr == :enabled
+        config_set('fex', 'feature_install', 'no')
       end
     rescue Cisco::CliError => e
       raise "[#{@name}] '#{e.command}' : #{e.clierror}"
     end
 
     def ipv4_addr_mask
-      @@node.config_get("interface", "ipv4_addr_mask", @name)
+      config_get('interface', 'ipv4_addr_mask', @name)
     end
 
     def ipv4_addr_mask_set(addr, mask)
       check_switchport_disabled
-      if addr.nil? or addr == default_ipv4_address
-        @@node.config_set("interface", "ipv4_addr_mask", @name, "no", "")
+      if addr.nil? || addr == default_ipv4_address
+        config_set('interface', 'ipv4_addr_mask', @name, 'no', '')
       else
-        @@node.config_set("interface", "ipv4_addr_mask", @name, "",
-                         "#{addr}/#{mask}")
+        config_set('interface', 'ipv4_addr_mask', @name, '',
+                   "#{addr}/#{mask}")
       end
     rescue Cisco::CliError => e
       raise "[#{@name}] '#{e.command}' : #{e.clierror}"
@@ -165,116 +166,115 @@ module Cisco
     def ipv4_address
       val = ipv4_addr_mask
       return default_ipv4_address if val.nil?
-      addr, mask = val.shift
-      addr
+      # val is [[addr, mask]] - we just want the addr
+      val.shift.first
     end
 
     def default_ipv4_address
-      @@node.config_get_default("interface", "ipv4_address")
+      config_get_default('interface', 'ipv4_address')
     end
 
     def ipv4_netmask_length
       val = ipv4_addr_mask
       return default_ipv4_netmask_length if val.nil?
-      addr, mask = val.shift
-      mask.to_i
+      # val is [[addr, mask]] - we just want the mask
+      val.shift.last.to_i
     end
 
     def default_ipv4_netmask_length
-      @@node.config_get_default("interface", "ipv4_netmask_length")
+      config_get_default('interface', 'ipv4_netmask_length')
     end
 
     def ipv4_proxy_arp
-      state = @@node.config_get("interface", "ipv4_proxy_arp", @name)
+      state = config_get('interface', 'ipv4_proxy_arp', @name)
       state.nil? ? false : true
     end
 
     def ipv4_proxy_arp=(proxy_arp)
       check_switchport_disabled
-      no_cmd = (proxy_arp ? "" : "no")
-      @@node.config_set("interface", "ipv4_proxy_arp", @name, no_cmd)
+      no_cmd = (proxy_arp ? '' : 'no')
+      config_set('interface', 'ipv4_proxy_arp', @name, no_cmd)
     end
 
     def default_ipv4_proxy_arp
-      @@node.config_get_default("interface", "ipv4_proxy_arp")
+      config_get_default('interface', 'ipv4_proxy_arp')
     end
 
     def ipv4_redirects_lookup_string
       case @name
       when /loopback/i
-        return "ipv4_redirects_loopback"
+        return 'ipv4_redirects_loopback'
       else
-        return "ipv4_redirects_other_interfaces"
+        return 'ipv4_redirects_other_interfaces'
       end
     end
 
     def ipv4_redirects
       begin
-        state = @@node.config_get("interface",
-                                  ipv4_redirects_lookup_string, @name)
+        state = config_get('interface',
+                           ipv4_redirects_lookup_string, @name)
       rescue IndexError
         state = nil
       end
       # We return default state for the platform if the platform doesn't support
       # the command
-      return default_ipv4_redirects if state.nil? or state.empty?
+      return default_ipv4_redirects if state.nil? || state.empty?
       state.shift[/^ip redirects$/] ? true : false
     end
 
     def ipv4_redirects=(redirects)
       check_switchport_disabled
-      no_cmd = (redirects ? "" : "no")
-      @@node.config_set("interface", ipv4_redirects_lookup_string, @name, no_cmd)
+      no_cmd = (redirects ? '' : 'no')
+      config_set('interface', ipv4_redirects_lookup_string, @name, no_cmd)
     rescue IndexError
       raise "ipv4 redirects not supported on #{@name}"
     end
 
     def default_ipv4_redirects
-      @@node.config_get_default("interface", ipv4_redirects_lookup_string)
+      config_get_default('interface', ipv4_redirects_lookup_string)
     end
 
     def feature_lacp?
-      not @@node.config_get("interface", "feature_lacp").nil?
+      !config_get('interface', 'feature_lacp').nil?
     end
 
     def feature_lacp_set(val)
       return if feature_lacp? == val
-      @@node.config_set("interface", "feature_lacp", val ? "" : "no")
+      config_set('interface', 'feature_lacp', val ? '' : 'no')
     end
 
     def mtu
-      mtu = @@node.config_get("interface", "mtu", @name)
+      mtu = config_get('interface', 'mtu', @name)
       return default_mtu if mtu.nil?
       mtu.shift.strip.to_i
     end
 
-    def mtu=(mtu)
-      mtu.nil? ?
-        @@node.config_set("interface", "mtu", @name, "no", "") :
-        @@node.config_set("interface", "mtu", @name, "", mtu)
+    def mtu=(val)
+      check_switchport_disabled
+      config_set('interface', 'mtu', @name, '', val)
     rescue Cisco::CliError => e
       raise "[#{@name}] '#{e.command}' : #{e.clierror}"
     end
 
     def default_mtu
-      @@node.config_get_default("interface", "mtu")
+      config_get_default('interface', 'mtu')
     end
 
     def negotiate_auto_lookup_string
       case @name
       when /Ethernet/i
-        return "negotiate_auto_ethernet"
+        return 'negotiate_auto_ethernet'
       when /port-channel/i # Ether-channel
-        return "negotiate_auto_portchannel"
+        return 'negotiate_auto_portchannel'
       else
-        return "negotiate_auto_other_interfaces"
+        return 'negotiate_auto_other_interfaces'
       end
     end
 
     def negotiate_auto
       lookup = negotiate_auto_lookup_string
       begin
-        state = @@node.config_get("interface", lookup, @name)
+        state = config_get('interface', lookup, @name)
       rescue IndexError
         # We return default state even if the config_get is not supported
         # for this platform / interface type. This is done so that we can set
@@ -287,9 +287,9 @@ module Cisco
 
     def negotiate_auto=(negotiate_auto)
       lookup = negotiate_auto_lookup_string
-      no_cmd = (negotiate_auto ? "" : "no")
+      no_cmd = (negotiate_auto ? '' : 'no')
       begin
-        @@node.config_set("interface", lookup, @name, no_cmd)
+        config_set('interface', lookup, @name, no_cmd)
       rescue Cisco::CliError => e
         raise "[#{@name}] '#{e.command}' : #{e.clierror}"
       rescue IndexError
@@ -298,17 +298,17 @@ module Cisco
     end
 
     def default_negotiate_auto
-      @@node.config_get_default("interface", negotiate_auto_lookup_string)
+      config_get_default('interface', negotiate_auto_lookup_string)
     end
 
     def shutdown
-      state = @@node.config_get("interface", "shutdown", @name)
+      state = config_get('interface', 'shutdown', @name)
       state ? true : false
     end
 
     def shutdown=(state)
-      no_cmd = (state ? "" : "no")
-      @@node.config_set("interface", "shutdown", @name, no_cmd)
+      no_cmd = (state ? '' : 'no')
+      config_set('interface', 'shutdown', @name, no_cmd)
     rescue Cisco::CliError => e
       raise "[#{@name}] '#{e.command}' : #{e.clierror}"
     end
@@ -319,75 +319,75 @@ module Cisco
         def_sw = system_default_switchport
         def_shut = system_default_switchport_shutdown
 
-        if def_sw and def_shut
-          lookup = "shutdown_ethernet_switchport_shutdown"
-        elsif def_sw and not def_shut
-          lookup = "shutdown_ethernet_switchport_noshutdown"
-        elsif not def_sw and def_shut
-          lookup = "shutdown_ethernet_noswitchport_shutdown"
-        elsif not def_sw and not def_shut
-          lookup = "shutdown_ethernet_noswitchport_noshutdown"
+        if def_sw && def_shut
+          lookup = 'shutdown_ethernet_switchport_shutdown'
+        elsif def_sw && !def_shut
+          lookup = 'shutdown_ethernet_switchport_noshutdown'
+        elsif !def_sw && def_shut
+          lookup = 'shutdown_ethernet_noswitchport_shutdown'
+        elsif !def_sw && !def_shut
+          lookup = 'shutdown_ethernet_noswitchport_noshutdown'
         else
-          raise "Error: def_sw #{def_sw}, def_shut #{def_shut}"
+          fail "Error: def_sw #{def_sw}, def_shut #{def_shut}"
         end
 
       when /loopback/i
-        lookup = "shutdown_loopback"
+        lookup = 'shutdown_loopback'
 
-      when /port-channel/i  # EtherChannel
-        lookup = "shutdown_ether_channel"
+      when /port-channel/i # EtherChannel
+        lookup = 'shutdown_ether_channel'
 
       when /Vlan/i
-        lookup = "shutdown_vlan"
+        lookup = 'shutdown_vlan'
 
       else
-        lookup = "shutdown_unknown"
+        lookup = 'shutdown_unknown'
       end
-      @@node.config_get_default("interface", lookup)
+      config_get_default('interface', lookup)
     end
 
     def switchport
       # This is "switchport", not "switchport mode"
-      sw = @@node.config_get("interface", "switchport", @name)
+      sw = config_get('interface', 'switchport', @name)
       sw.nil? ? false : true
     end
 
     def switchport_enable(val=true)
-      @@node.config_set("interface", "switchport", @name, val ? "" : "no")
+      config_set('interface', 'switchport', @name, val ? '' : 'no')
     end
 
     # switchport_autostate_exclude is exclusive to switchport interfaces
     def switchport_autostate_exclude
-      not @@node.config_get("interface",
-                           "switchport_autostate_exclude", @name).nil?
+      !config_get('interface',
+                  'switchport_autostate_exclude', @name).nil?
     end
 
     def switchport_autostate_exclude=(val)
       # cannot configure autostate unless feature vlan is enabled
-      raise "switchport mode must be configured before switchport autostate" unless
-        switchport
+      fail('switchport mode must be configured before ' \
+           'switchport autostate') unless switchport
       feature_vlan_set(true)
-      @@node.config_set("interface", "switchport_autostate_exclude",
-                       @name, val ? "" : "no")
+      config_set('interface', 'switchport_autostate_exclude',
+                 @name, val ? '' : 'no')
     end
 
     def default_switchport_autostate_exclude
-      @@node.config_get_default("interface", "switchport_autostate_exclude")
+      config_get_default('interface', 'switchport_autostate_exclude')
     end
 
     def switchport_mode_lookup_string
       case @name
       when /Ethernet/i
-        return "switchport_mode_ethernet"
+        return 'switchport_mode_ethernet'
       when /port-channel/i
-        return "switchport_mode_port_channel"
+        return 'switchport_mode_port_channel'
       else
-        return "switchport_mode_other_interfaces"
+        return 'switchport_mode_other_interfaces'
       end
     end
 
     def switchport_mode
-      mode = @@node.config_get("interface", switchport_mode_lookup_string, @name)
+      mode = config_get('interface', switchport_mode_lookup_string, @name)
 
       return mode.nil? ? :disabled : IF_SWITCHPORT_MODE.key(mode.shift)
 
@@ -404,8 +404,8 @@ module Cisco
       if (:fex_fabric == mode_set)
         fex_feature_set(:enabled) unless (:enabled == fex_feature)
       end
-      @@node.config_set("interface", switchport_mode_lookup_string, @name, "",
-                       IF_SWITCHPORT_MODE[mode_set])
+      config_set('interface', switchport_mode_lookup_string, @name, '',
+                 IF_SWITCHPORT_MODE[mode_set])
 
     rescue RuntimeError
       raise "[#{@name}] switchport_mode is not supported on this interface"
@@ -416,18 +416,18 @@ module Cisco
       # int e1/1
       #   switchport
       #   switchport mode [access|trunk|fex|...]
-      raise ArgumentError unless IF_SWITCHPORT_MODE.keys.include? mode_set
+      fail ArgumentError unless IF_SWITCHPORT_MODE.keys.include? mode_set
       case mode_set
       when :disabled
         if switchport
           # Note: turn off switchport command, not switchport mode
-          @@node.config_set("interface", "switchport", @name, "no")
+          config_set('interface', 'switchport', @name, 'no')
         end
 
       when :default
         if :disabled == default_switchport_mode
-          @@node.config_set("interface", switchport_mode_lookup_string,
-                            @name, "no", "")
+          config_set('interface', switchport_mode_lookup_string,
+                     @name, 'no', '')
         else
           switchport_enable_and_mode(mode_set)
         end
@@ -443,104 +443,108 @@ module Cisco
     def default_switchport_mode
       return :disabled unless system_default_switchport
       IF_SWITCHPORT_MODE.key(
-        @@node.config_get_default("interface", switchport_mode_lookup_string))
+        config_get_default('interface', switchport_mode_lookup_string))
     end
 
     def switchport_trunk_allowed_vlan
-      val = @@node.config_get(
-        "interface", "switchport_trunk_allowed_vlan", @name)
+      val = config_get(
+        'interface', 'switchport_trunk_allowed_vlan', @name)
       return default_switchport_trunk_allowed_vlan if val.nil?
       val.shift.strip
     end
 
     def switchport_trunk_allowed_vlan=(val)
-      val.nil? ?
-        @@node.config_set(
-          "interface", "switchport_trunk_allowed_vlan", @name, "no", "") :
-        @@node.config_set(
-          "interface", "switchport_trunk_allowed_vlan", @name, "", val)
+      if val.nil?
+        config_set(
+          'interface', 'switchport_trunk_allowed_vlan', @name, 'no', '')
+      else
+        config_set(
+          'interface', 'switchport_trunk_allowed_vlan', @name, '', val)
+      end
     rescue Cisco::CliError => e
       raise "[#{@name}] '#{e.command}' : #{e.clierror}"
     end
 
     def default_switchport_trunk_allowed_vlan
-      @@node.config_get_default("interface", "switchport_trunk_allowed_vlan")
+      config_get_default('interface', 'switchport_trunk_allowed_vlan')
     end
 
     def switchport_trunk_native_vlan
-      val = @@node.config_get(
-        "interface", "switchport_trunk_native_vlan", @name)
+      val = config_get(
+        'interface', 'switchport_trunk_native_vlan', @name)
       return default_switchport_trunk_native_vlan if val.nil?
       val.shift.strip.to_i
     end
 
     def switchport_trunk_native_vlan=(val)
-      val.nil? ?
-        @@node.config_set(
-          "interface", "switchport_trunk_native_vlan", @name, "no", "") :
-        @@node.config_set(
-          "interface", "switchport_trunk_native_vlan", @name, "", val)
+      if val.nil?
+        config_set(
+          'interface', 'switchport_trunk_native_vlan', @name, 'no', '')
+      else
+        config_set(
+          'interface', 'switchport_trunk_native_vlan', @name, '', val)
+      end
     rescue Cisco::CliError => e
       raise "[#{@name}] '#{e.command}' : #{e.clierror}"
     end
 
     def default_switchport_trunk_native_vlan
-      @@node.config_get_default("interface", "switchport_trunk_native_vlan")
+      config_get_default('interface', 'switchport_trunk_native_vlan')
     end
 
     def system_default_switchport
       # This command is a user-configurable system default.
-      sys_def = @@node.config_get("interface", "system_default_switchport")
+      sys_def = config_get('interface', 'system_default_switchport')
       sys_def.nil? ? false : true
     end
 
     def system_default_switchport_shutdown
       # This command is a user-configurable system default.
-      sys_def = @@node.config_get("interface",
-                                  "system_default_switchport_shutdown")
+      sys_def = config_get('interface',
+                           'system_default_switchport_shutdown')
       sys_def.nil? ? false : true
     end
 
     def system_default_svi_autostate
       # This command is a user-configurable system default.
-      sys_def = @@node.config_get("interface",
-                                  "system_default_svi_autostate")
+      sys_def = config_get('interface',
+                           'system_default_svi_autostate')
       sys_def.nil? ? false : true
     end
 
     def switchport_vtp_mode_capable?
-      not switchport_mode.to_s.match(/(access|trunk)/).nil?
+      !switchport_mode.to_s.match(/(access|trunk)/).nil?
     end
 
     def switchport_vtp
       return false unless switchport_vtp_mode_capable?
-      vtp = @@node.config_get("interface", "vtp", @name)
+      vtp = config_get('interface', 'vtp', @name)
       vtp.nil? ? false : true
     end
 
     def switchport_vtp=(vtp_set)
       return false unless switchport_vtp_mode_capable?
-      no_cmd = (vtp_set) ? "" : "no"
-      @@node.config_set("interface", "vtp", @name, no_cmd)
+      no_cmd = (vtp_set) ? '' : 'no'
+      config_set('interface', 'vtp', @name, no_cmd)
     rescue Cisco::CliError => e
       raise "[#{@name}] '#{e.command}' : #{e.clierror}"
     end
 
     def svi_cmd_allowed?(cmd)
-      raise "[#{@name}] Invalid interface type for command [#{cmd}]" unless
+      fail "[#{@name}] Invalid interface type for command [#{cmd}]" unless
         @name[/vlan/i]
     end
 
     # svi_autostate is exclusive to svi interfaces
     def svi_autostate
       return nil unless @name[/^vlan/i]
-      not @@node.config_get("interface", "svi_autostate", @name).nil?
+      !config_get('interface', 'svi_autostate', @name).nil?
     end
 
     def svi_autostate=(val)
       check_switchport_disabled
       svi_cmd_allowed?('autostate')
-      @@node.config_set("interface", "svi_autostate", @name, val ? "" : "no")
+      config_set('interface', 'svi_autostate', @name, val ? '' : 'no')
     end
 
     def default_svi_autostate
@@ -548,60 +552,62 @@ module Cisco
     end
 
     def feature_vlan?
-      not @@node.config_get("interface", "feature_vlan").nil?
+      !config_get('interface', 'feature_vlan').nil?
     end
 
     def feature_vlan_set(val)
       return if feature_vlan? == val
-      @@node.config_set("interface", "feature_vlan", val ? "" : "no")
+      config_set('interface', 'feature_vlan', val ? '' : 'no')
     end
 
     # svi_management is exclusive to svi interfaces
     def svi_management
       return nil unless @name[/^vlan/i]
-      not @@node.config_get("interface", "svi_management", @name).nil?
+      !config_get('interface', 'svi_management', @name).nil?
     end
 
     def svi_management=(val)
       check_switchport_disabled
       svi_cmd_allowed?('management')
-      @@node.config_set("interface", "svi_management", @name, val ? "" : "no")
+      config_set('interface', 'svi_management', @name, val ? '' : 'no')
     end
 
     def default_svi_management
-      @@node.config_get_default("interface", "svi_management")
+      config_get_default('interface', 'svi_management')
     end
 
     def default_switchport_vtp
-      @@node.config_get_default("interface", "vtp")
+      config_get_default('interface', 'vtp')
     end
 
     def switchport_vtp_feature?
-      @@node.config_get("vtp", "feature")
+      config_get('vtp', 'feature')
     end
 
     def check_switchport_disabled
-      raise "#{caller[0][/`.*'/][1..-2]} cannot be set unless switchport mode" +
-        " is disabled" unless switchport_mode == :disabled
+      fail "#{caller[0][/`.*'/][1..-2]} cannot be set unless switchport mode" \
+        ' is disabled' unless switchport_mode == :disabled
     end
 
     def vrf
-      vrf = @@node.config_get("interface", "vrf", @name)
-      return "" if vrf.nil?
+      vrf = config_get('interface', 'vrf', @name)
+      return '' if vrf.nil?
       vrf.shift.strip
     end
 
     def vrf=(vrf)
-      raise TypeError unless vrf.is_a?(String)
-      vrf.empty? ?
-        @@node.config_set("interface", "vrf", @name, "no", "") :
-        @@node.config_set("interface", "vrf", @name, "", vrf)
+      fail TypeError unless vrf.is_a?(String)
+      if vrf.empty?
+        config_set('interface', 'vrf', @name, 'no', '')
+      else
+        config_set('interface', 'vrf', @name, '', vrf)
+      end
     rescue Cisco::CliError => e
       raise "[#{@name}] '#{e.command}' : #{e.clierror}"
     end
 
     def default_vrf
-      @@node.config_get_default("interface", "vrf")
+      config_get_default('interface', 'vrf')
     end
   end  # Class
 end    # Module