Support 'local' routes with redhat provider

Author: treydock

README.md

@@ -80,6 +80,14 @@ network_route { 'default':
   netmask   => '0.0.0.0',
   network   => 'default'
 }
+network_route { '10.0.0.2':
+  ensure    => 'present',
+  gateway   => 'local',
+  netmask   => 'local',
+  network   => 'local',
+  interface => 'eth0',
+  options   => 'proto 66 scope host table local',
+}
 ```
 
   For SLES:

lib/puppet/provider/network_route/redhat.rb

@@ -42,13 +42,25 @@ def self.parse_file(_filename, contents)
       route = line.split(' ', 6)
       raise Puppet::Error, 'Malformed redhat route file, cannot instantiate network_route resources' if route.length < 4
 
+      route = line.split(' ', 5) if route[0] == 'local'
+
       new_route = {}
 
-      new_route[:gateway] = route[2]
-      new_route[:interface] = route[4]
-      new_route[:options] = route[5] if route[5]
+      if route[0] == 'local'
+        new_route[:gateway] = 'local'
+        new_route[:interface] = route[3]
+        new_route[:options] = route[4] if route[4]
+      else
+        new_route[:gateway] = route[2]
+        new_route[:interface] = route[4]
+        new_route[:options] = route[5] if route[5]
+      end
 
-      if ['default', '0.0.0.0'].include? route[0]
+      if route[0] == 'local'
+        new_route[:name] = route[1]
+        new_route[:network] = 'local'
+        new_route[:netmask] = 'local'
+      elsif ['default', '0.0.0.0'].include? route[0]
         new_route[:name]    = 'default' # FIXME: Must match :name in order for changes to be detected
         new_route[:network] = 'default'
         new_route[:netmask] = '0.0.0.0'
@@ -77,6 +89,8 @@ def self.format_file(_filename, providers)
       end
       contents << if provider.network == 'default'
                     "#{provider.network} via #{provider.gateway} dev #{provider.interface}"
+                  elsif provider.network == 'local'
+                    "#{provider.network} #{provider.name} dev #{provider.interface}"
                   else
                     ip = IPAddr.new("#{provider.network}/#{provider.netmask}")
                     "#{ip}/#{ip.prefix} via #{provider.gateway} dev #{provider.interface}"

lib/puppet/type/network_route.rb

@@ -22,7 +22,7 @@
     isrequired
     desc 'The target network address'
     validate do |value|
-      unless value == 'default'
+      unless %w[default local].include?(value)
         a = PuppetX::Voxpupuli::Utils.try { IPAddr.new(value) }
         raise("Invalid value for parameter 'network': #{value}") unless a
       end
@@ -34,12 +34,14 @@
     desc 'The subnet mask to apply to the route'
 
     validate do |value|
-      raise("Invalid value for parameter 'netmask': #{value}") unless value.length <= 3 || PuppetX::Voxpupuli::Utils.try { IPAddr.new(value) }
+      raise("Invalid value for parameter 'netmask': #{value}") unless value == 'local' || value.length <= 3 || PuppetX::Voxpupuli::Utils.try { IPAddr.new(value) }
     end
 
     munge do |value|
+      if value == 'local'
+        value
       # '255.255.255.255'.to_i  will return 255, so we try to convert it back:
-      if value.to_i.to_s == value
+      elsif value.to_i.to_s == value
         # what are the chances someone is using /16 for their IPv6 network?
         addr = value.to_i <= 32 ? '255.255.255.255' : 'ffff:ffff:ffff:ffff:ffff:ffff:ffff:ffff'
         IPAddr.new(addr).mask(value.strip.to_i).to_s
@@ -58,7 +60,7 @@
     desc 'The gateway to use for the route'
 
     validate do |value|
-      IPAddr.new(value)
+      IPAddr.new(value) unless value == 'local'
     rescue ArgumentError
       raise("Invalid value for parameter 'gateway': #{value}")
     end

spec/fixtures/provider/network_route/redhat/local_routes

@@ -0,0 +1,3 @@
+10.0.0.0/8 via 10.0.0.1 dev eth0
+192.168.0.0/16 via 10.0.0.1 table n0-hc
+local 10.0.0.2 dev eth0 proto 66 scope host table local

spec/unit/provider/network_route/redhat_spec.rb

@@ -69,6 +69,23 @@ def fixture_data(file)
       end
     end
 
+    describe 'an advanced file using local route' do
+      let :data do
+        described_class.parse_file('', fixture_data('local_routes'))
+      end
+
+      it 'parses out normal ipv4 network routes' do
+        expect(data.find { |h| h[:name] == '10.0.0.2' }).to eq(
+          name: '10.0.0.2',
+          network: 'local',
+          netmask: 'local',
+          gateway: 'local',
+          interface: 'eth0',
+          options: 'proto 66 scope host table local'
+        )
+      end
+    end
+
     describe 'an invalid file' do
       it 'fails' do
         expect do
@@ -127,8 +144,20 @@ def fixture_data(file)
       )
     end
 
+    let :local_provider do
+      instance_double(
+        'local_provider',
+        name: '10.0.0.2',
+        network: 'local',
+        netmask: '',
+        gateway: 'local',
+        interface: 'eth0',
+        options: 'proto 66 scope host table local'
+      )
+    end
+
     let :content do
-      described_class.format_file('', [route1_provider, route2_provider, defaultroute_provider, nooptions_provider])
+      described_class.format_file('', [route1_provider, route2_provider, defaultroute_provider, nooptions_provider, local_provider])
     end
 
     describe 'writing the route line' do
@@ -166,5 +195,11 @@ def fixture_data(file)
         expect(content).not_to match(%r{absent})
       end
     end
+
+    describe 'for local routes' do
+      it 'has local route defined' do
+        expect(content.scan(%r{^local .*$}).first).to include('local 10.0.0.2 dev eth0 proto 66 scope host table local')
+      end
+    end
   end
 end

spec/unit/type/network_route_spec.rb

@@ -35,6 +35,11 @@
 
   describe 'when validating the attribute value' do
     describe 'network' do
+      it 'allows local' do
+        r = Puppet::Type.type(:network_route).new(name: '192.168.1.0/24', network: 'local', netmask: '255.255.255.0', gateway: '23.23.23.42', interface: 'eth0')
+        expect(r[:network]).to eq('local')
+      end
+
       it 'validates the network as an IP address' do
         expect do
           Puppet::Type.type(:network_route).new(name: '192.168.1.0/24', network: 'not an ip address', netmask: '255.255.255.0', gateway: '23.23.23.42', interface: 'eth0')
@@ -63,9 +68,19 @@
         r = Puppet::Type.type(:network_route).new(name: '192.168.1.0/24', network: '192.168.1.0', netmask: '255.255.128.0', gateway: '23.23.23.42', interface: 'eth0')
         expect(r[:netmask]).to eq('255.255.128.0')
       end
+
+      it 'allows local' do
+        r = Puppet::Type.type(:network_route).new(name: '192.168.1.0/24', network: '192.168.1.0', netmask: 'local', gateway: '23.23.23.42', interface: 'eth0')
+        expect(r[:netmask]).to eq('local')
+      end
     end
 
     describe 'gateway' do
+      it 'allows local' do
+        r = Puppet::Type.type(:network_route).new(name: '192.168.1.0/24', network: '192.168.1.0', netmask: '255.255.255.0', gateway: 'local', interface: 'eth0')
+        expect(r[:gateway]).to eq('local')
+      end
+
       it 'validates as an IP address' do
         expect do
           Puppet::Type.type(:network_route).new(name: '192.168.1.0/24', network: '192.168.1.0', netmask: '255.255.255.0', gateway: 'not an ip address', interface: 'eth0')