Merge pull request #1940 from puppetlabs/FACT-2679

FACT-2679 Get DHCP for all interfaces on OSX

Author: Filipovici-Andrei

lib/facts/linux/dhcp_servers.rb

@@ -14,11 +14,11 @@ def call_the_resolver
       private
 
       def construct_addresses_hash
-        primary = Facter::Resolvers::NetworkingLinux.resolve(:primary_interface)
+        primary_dhcp = Facter::Resolvers::NetworkingLinux.resolve(:dhcp)
         interfaces = Facter::Resolvers::NetworkingLinux.resolve(:interfaces)
         return unless interfaces
 
-        servers = { system: interfaces[primary][:dhcp] }
+        servers = { system: primary_dhcp }
         interfaces&.each { |interface_name, info| servers[interface_name] = info[:dhcp] if info[:dhcp] }
         servers
       end

lib/facts/linux/networking/dhcp.rb

@@ -7,10 +7,7 @@ class Dhcp
         FACT_NAME = 'networking.dhcp'
 
         def call_the_resolver
-          interfaces = Facter::Resolvers::NetworkingLinux.resolve(:interfaces)
-          primary = Facter::Resolvers::NetworkingLinux.resolve(:primary_interface)
-
-          fact_value = interfaces[primary][:dhcp] if interfaces && interfaces[primary]
+          fact_value = Facter::Resolvers::NetworkingLinux.resolve(:dhcp)
 
           Facter::ResolvedFact.new(FACT_NAME, fact_value)
         end

lib/facts/linux/networking/domain.rb

@@ -9,6 +9,7 @@ class Domain
 
         def call_the_resolver
           fact_value = Facter::Resolvers::Hostname.resolve(:domain)
+
           [Facter::ResolvedFact.new(FACT_NAME, fact_value), Facter::ResolvedFact.new(ALIASES, fact_value, :legacy)]
         end
       end

lib/facts/linux/networking/fqdn.rb

@@ -9,6 +9,7 @@ class Fqdn
 
         def call_the_resolver
           fact_value = Facter::Resolvers::Hostname.resolve(:fqdn)
+
           [Facter::ResolvedFact.new(FACT_NAME, fact_value), Facter::ResolvedFact.new(ALIASES, fact_value, :legacy)]
         end
       end

lib/facts/linux/networking/hostname.rb

@@ -9,6 +9,7 @@ class Hostname
 
         def call_the_resolver
           fact_value = Facter::Resolvers::Hostname.resolve(:hostname)
+
           [Facter::ResolvedFact.new(FACT_NAME, fact_value), Facter::ResolvedFact.new(ALIASES, fact_value, :legacy)]
         end
       end

lib/facts/linux/networking/interfaces.rb

@@ -8,6 +8,7 @@ class Interfaces
 
         def call_the_resolver
           fact_value = Facter::Resolvers::NetworkingLinux.resolve(:interfaces)
+
           Facter::ResolvedFact.new(FACT_NAME, fact_value)
         end
       end

lib/facts/linux/networking/ip.rb

@@ -8,10 +8,7 @@ class Ip
         ALIASES = 'ipaddress'
 
         def call_the_resolver
-          interfaces = Facter::Resolvers::NetworkingLinux.resolve(:interfaces)
-          primary = Facter::Resolvers::NetworkingLinux.resolve(:primary_interface)
-
-          fact_value = interfaces[primary][:ip] if interfaces && interfaces[primary]
+          fact_value = Facter::Resolvers::NetworkingLinux.resolve(:ip)
 
           [Facter::ResolvedFact.new(FACT_NAME, fact_value), Facter::ResolvedFact.new(ALIASES, fact_value, :legacy)]
         end

lib/facts/linux/networking/ip6.rb

@@ -8,10 +8,7 @@ class Ip6
         ALIASES = 'ipaddress6'
 
         def call_the_resolver
-          interfaces = Facter::Resolvers::NetworkingLinux.resolve(:interfaces)
-          primary = Facter::Resolvers::NetworkingLinux.resolve(:primary_interface)
-
-          fact_value = interfaces[primary][:ip6] if interfaces && interfaces[primary]
+          fact_value = Facter::Resolvers::NetworkingLinux.resolve(:ip6)
 
           [Facter::ResolvedFact.new(FACT_NAME, fact_value), Facter::ResolvedFact.new(ALIASES, fact_value, :legacy)]
         end

lib/facts/linux/networking/mac.rb

@@ -8,10 +8,7 @@ class Mac
         ALIASES = 'macaddress'
 
         def call_the_resolver
-          interfaces = Facter::Resolvers::NetworkingLinux.resolve(:interfaces)
-          primary = Facter::Resolvers::NetworkingLinux.resolve(:primary_interface)
-
-          fact_value = interfaces[primary][:mac] if interfaces && interfaces[primary]
+          fact_value = Facter::Resolvers::NetworkingLinux.resolve(:mac)
 
           [Facter::ResolvedFact.new(FACT_NAME, fact_value), Facter::ResolvedFact.new(ALIASES, fact_value, :legacy)]
         end

lib/facts/linux/networking/mtu.rb

@@ -7,10 +7,7 @@ class Mtu
         FACT_NAME = 'networking.mtu'
 
         def call_the_resolver
-          interfaces = Facter::Resolvers::NetworkingLinux.resolve(:interfaces)
-          primary = Facter::Resolvers::NetworkingLinux.resolve(:primary_interface)
-
-          fact_value = interfaces[primary][:mtu] if interfaces && interfaces[primary]
+          fact_value = Facter::Resolvers::NetworkingLinux.resolve(:mtu)
 
           Facter::ResolvedFact.new(FACT_NAME, fact_value)
         end