ohai 14.2.0 → 14.3.0

data/lib/ohai/plugins/darwin/platform.rb

@@ -20,7 +20,7 @@ Ohai.plugin(:Platform) do
   provides "platform", "platform_version", "platform_build", "platform_family"
 
   collect_data(:darwin) do
-    so = shell_out("#{Ohai.abs_path( "/usr/bin/sw_vers" )}")
+    so = shell_out((Ohai.abs_path( "/usr/bin/sw_vers" )).to_s)
     so.stdout.lines do |line|
       case line
       when /^ProductName:\s+(.+)$/

data/lib/ohai/plugins/dmi.rb

@@ -47,8 +47,8 @@ Ohai.plugin(:DMI) do
     begin
       so = shell_out("dmidecode")
       # ==== EXAMPLE RECORD: ====
-      #Handle 0x0000, DMI type 0, 24 bytes
-      #BIOS Information
+      # Handle 0x0000, DMI type 0, 24 bytes
+      # BIOS Information
       #        Vendor: American Megatrends Inc.
       #        Version: 080012
       # ... similar lines trimmed
@@ -58,7 +58,7 @@ Ohai.plugin(:DMI) do
       # ... similar lines trimmed
       so.stdout.lines do |line|
         next if blank_line.match(line)
-        line = line.encode(line.encoding, :universal_newline => true)
+        line = line.encode(line.encoding, universal_newline: true)
 
         if ( dmidecode_version = dmidecode_version_line.match(line) )
           dmi[:dmidecode_version] = dmidecode_version[1]
@@ -80,10 +80,10 @@ Ohai.plugin(:DMI) do
             next
           end
 
-          dmi_record = { :type => Ohai::Common::DMI.id_lookup(handle[2]) }
+          dmi_record = { type: Ohai::Common::DMI.id_lookup(handle[2]) }
 
-          dmi[dmi_record[:type]] = Mash.new unless dmi.has_key?(dmi_record[:type])
-          dmi[dmi_record[:type]][:all_records] = [] unless dmi[dmi_record[:type]].has_key?(:all_records)
+          dmi[dmi_record[:type]] = Mash.new unless dmi.key?(dmi_record[:type])
+          dmi[dmi_record[:type]][:all_records] = [] unless dmi[dmi_record[:type]].key?(:all_records)
           dmi_record[:position] = dmi[dmi_record[:type]][:all_records].length
           dmi[dmi_record[:type]][:all_records].push(Mash.new)
           dmi[dmi_record[:type]][:all_records][dmi_record[:position]][:record_id] = handle[1]

data/lib/ohai/plugins/filesystem.rb

@@ -39,12 +39,12 @@ Ohai.plugin(:Filesystem) do
         uuid = $2
         label = $3
         fs_type = $4
-        return { :dev => dev, :uuid => uuid, :label => label, :fs_type => fs_type }
+        return { dev: dev, uuid: uuid, label: label, fs_type: fs_type }
       end
     when "blkid"
       bits = line.split
       dev = bits.shift.split(":")[0]
-      f = { :dev => dev }
+      f = { dev: dev }
       bits.each do |keyval|
         if keyval =~ /(\S+)="(\S+)"/
           key = $1.downcase.to_sym
@@ -151,7 +151,7 @@ Ohai.plugin(:Filesystem) do
       so.stdout.each_line do |line|
         if line =~ /^(.+?) on (.+?) type (.+?) \((.+?)\)$/
           key = "#{$1},#{$2}"
-          fs[key] = Mash.new unless fs.has_key?(key)
+          fs[key] = Mash.new unless fs.key?(key)
           fs[key][:device] = $1
           fs[key][:mount] = $2
           fs[key][:fs_type] = $3
@@ -237,7 +237,7 @@ Ohai.plugin(:Filesystem) do
       mounts.each_line do |line|
         if line =~ /^(\S+) (\S+) (\S+) (\S+) \S+ \S+$/
           key = "#{$1},#{$2}"
-          next if fs.has_key?(key)
+          next if fs.key?(key)
           fs[key] = Mash.new
           fs[key][:device] = $1
           fs[key][:mount] = $2
@@ -360,7 +360,7 @@ Ohai.plugin(:Filesystem) do
     so.stdout.lines do |line|
       if line =~ /^(.+?) on (.+?) \((.+?), (.+?)\)$/
         key = "#{$1},#{$2}"
-        fs[key] = Mash.new unless fs.has_key?(key)
+        fs[key] = Mash.new unless fs.key?(key)
         fs[key][:mount] = $2
         fs[key][:fs_type] = $3
         fs[key][:mount_options] = $4.split(/,\s*/)

data/lib/ohai/plugins/hostname.rb

@@ -6,7 +6,7 @@
 # Author:: Doug MacEachern (<dougm@vmware.com>)
 # Author:: James Gartrell (<jgartrel@gmail.com>)
 # Author:: Isa Farnik (<isa@chef.io>)
-# Copyright:: Copyright (c) 2008-2016 Chef Software, Inc.
+# Copyright:: Copyright (c) 2008-2018, Chef Software Inc.
 # Copyright:: Copyright (c) 2009 Bryan McLellan
 # Copyright:: Copyright (c) 2009 Daniel DeLeo
 # Copyright:: Copyright (c) 2010 VMware, Inc.
@@ -166,15 +166,15 @@ Ohai.plugin(:Hostname) do
     wmi = WmiLite::Wmi.new
     host = wmi.first_of("Win32_ComputerSystem")
 
-    hostname "#{host['dnshostname']}"
-    machinename "#{host['name']}"
+    hostname host["dnshostname"].to_s
+    machinename host["name"].to_s
 
     info = Socket.gethostbyname(Socket.gethostname)
     if info.first =~ /.+?\.(.*)/
       fqdn info.first
     else
-      #host is not in dns. optionally use:
-      #C:\WINDOWS\system32\drivers\etc\hosts
+      # host is not in dns. optionally use:
+      # C:\WINDOWS\system32\drivers\etc\hosts
       fqdn Socket.gethostbyaddr(info.last).first
     end
     domain collect_domain

data/lib/ohai/plugins/kernel.rb

@@ -42,11 +42,11 @@ Ohai.plugin(:Kernel) do
   # @return [Mash]
   def bsd_modules(path)
     modules = Mash.new
-    so = shell_out("#{Ohai.abs_path(path)}")
+    so = shell_out((Ohai.abs_path(path)).to_s)
     so.stdout.lines do |line|
       #  1    7 0xc0400000 97f830   kernel
       if line =~ /(\d+)\s+(\d+)\s+([0-9a-fx]+)\s+([0-9a-fx]+)\s+([a-zA-Z0-9\_]+)/
-        modules[$5] = { :size => $4, :refcount => $2 }
+        modules[$5] = { size: $4, refcount: $2 }
       end
     end
     modules
@@ -169,7 +169,7 @@ Ohai.plugin(:Kernel) do
     so = shell_out("kextstat -k -l")
     so.stdout.lines do |line|
       if line =~ /(\d+)\s+(\d+)\s+0x[0-9a-f]+\s+0x([0-9a-f]+)\s+0x[0-9a-f]+\s+([a-zA-Z0-9\.]+) \(([0-9\.]+)\)/
-        modules[$4] = { :version => $5, :size => $3.hex, :index => $1, :refcount => $2 }
+        modules[$4] = { version: $5, size: $3.hex, index: $1, refcount: $2 }
       end
     end
 
@@ -198,7 +198,7 @@ Ohai.plugin(:Kernel) do
     so = shell_out("env lsmod")
     so.stdout.lines do |line|
       if line =~ /([a-zA-Z0-9\_]+)\s+(\d+)\s+(\d+)/
-        modules[$1] = { :size => $2, :refcount => $3 }
+        modules[$1] = { size: $2, refcount: $3 }
         # Making sure to get the module version that has been loaded
         if File.exist?("/sys/module/#{$1}/version")
           version = File.read("/sys/module/#{$1}/version").chomp.strip
@@ -239,7 +239,7 @@ Ohai.plugin(:Kernel) do
     module_description = /[\s]*([\d]+)[\s]+([a-f\d]+)[\s]+([a-f\d]+)[\s]+(?:[\-\d]+)[\s]+(?:[\d]+)[\s]+([\S]+)[\s]+\((.+)\)$/
     so.stdout.lines do |line|
       if ( mod = module_description.match(line) )
-        modules[mod[4]] = { :id => mod[1].to_i, :loadaddr => mod[2], :size => mod[3].to_i(16), :description => mod[5] }
+        modules[mod[4]] = { id: mod[1].to_i, loadaddr: mod[2], size: mod[3].to_i(16), description: mod[5] }
       end
     end
 
@@ -263,8 +263,8 @@ Ohai.plugin(:Kernel) do
       kernel[:os_info][p.name.wmi_underscore.to_sym] = host[p.name.downcase]
     end
 
-    kernel[:name] = "#{kernel[:os_info][:caption]}"
-    kernel[:release] = "#{kernel[:os_info][:version]}"
+    kernel[:name] = (kernel[:os_info][:caption]).to_s
+    kernel[:release] = (kernel[:os_info][:version]).to_s
     kernel[:version] = "#{kernel[:os_info][:version]} #{kernel[:os_info][:csd_version]} Build #{kernel[:os_info][:build_number]}"
     kernel[:os] = os_type_decode(kernel[:os_info][:os_type]) || languages[:ruby][:host_os]
     kernel[:product_type] = product_type_decode(kernel[:os_info][:product_type])
@@ -277,7 +277,7 @@ Ohai.plugin(:Kernel) do
       kernel[:cs_info][p.name.wmi_underscore.to_sym] = host[p.name.downcase]
     end
 
-    kernel[:machine] = arch_lookup("#{kernel[:cs_info][:system_type]}")
+    kernel[:machine] = arch_lookup((kernel[:cs_info][:system_type]).to_s)
     kernel[:system_type] = pc_system_type_decode(kernel[:cs_info][:pc_system_type])
   end
 end

data/lib/ohai/plugins/linux/lspci.rb

@@ -26,11 +26,11 @@ Ohai.plugin(:Lspci) do
     devices = Mash.new
     lspci = shell_out("lspci -vnnmk")
 
-    h = /[0-9a-fA-F]/ #any hex digit
-    hh = /#{h}#{h}/ #any 2 hex digits
-    hhhh = /#{h}#{h}#{h}#{h}/ #any 4 hex digits
+    h = /[0-9a-fA-F]/ # any hex digit
+    hh = /#{h}#{h}/ # any 2 hex digits
+    hhhh = /#{h}#{h}#{h}#{h}/ # any 4 hex digits
 
-    d_id = String.new #This identifies our pci devices
+    d_id = String.new # This identifies our pci devices
 
     def standard_form(devices, d_id, hhhh, tag, line)
       tmp = line.scan(/(.*)\s\[(#{hhhh})\]/)[0]

data/lib/ohai/plugins/linux/memory.rb

@@ -85,13 +85,13 @@ Ohai.plugin(:Memory) do
       when /^SwapFree:\s+(\d+) (.+)$/
         memory[:swap][:free] = "#{$1}#{$2}"
       when /^HugePages_Total:\s+(\d+)$/
-        memory[:hugepages][:total] = "#{$1}"
+        memory[:hugepages][:total] = $1.to_s
       when /^HugePages_Free:\s+(\d+)$/
-        memory[:hugepages][:free] = "#{$1}"
+        memory[:hugepages][:free] = $1.to_s
       when /^HugePages_Rsvd:\s+(\d+)$/
-        memory[:hugepages][:reserved] = "#{$1}"
+        memory[:hugepages][:reserved] = $1.to_s
       when /^HugePages_Surp:\s+(\d+)$/
-        memory[:hugepages][:surplus] = "#{$1}"
+        memory[:hugepages][:surplus] = $1.to_s
       when /^Hugepagesize:\s+(\d+) (.+)$/
         memory[:hugepage_size] = "#{$1}#{$2}"
       end

data/lib/ohai/plugins/linux/network.rb

@@ -101,8 +101,8 @@ Ohai.plugin(:Network) do
           next
         end
 
-        route_entry = Mash.new(:destination => route_dest,
-                               :family => family[:name])
+        route_entry = Mash.new(destination: route_dest,
+                               family: family[:name])
         %w{via scope metric proto src}.each do |k|
           # http://rubular.com/r/pwTNp65VFf
           route_entry[k] = $1 if route_ending =~ /\b#{k}\s+([^\s]+)/
@@ -113,7 +113,7 @@ Ohai.plugin(:Network) do
         # unless the interface has no addresses of this type at all
         if route_entry[:src]
           addr = iface[route_int][:addresses]
-          unless addr.nil? || addr.has_key?(route_entry[:src]) ||
+          unless addr.nil? || addr.key?(route_entry[:src]) ||
               addr.values.all? { |a| a["family"] != family[:name] }
             logger.trace("Plugin Network: Skipping route entry whose src does not match the interface IP")
             next
@@ -140,7 +140,7 @@ Ohai.plugin(:Network) do
     iface.collect do |i, iv|
       if iv[:routes]
         iv[:routes].collect do |r|
-          r.merge(:dev => i) if r[:family] == family[:name]
+          r.merge(dev: i) if r[:family] == family[:name]
         end.compact
       end
     end.compact.flatten
@@ -388,9 +388,9 @@ Ohai.plugin(:Network) do
   def choose_default_route(routes)
     routes.select do |r|
       r[:destination] == "default"
-    end.sort do |x, y|
+    end.min do |x, y|
       (x[:metric].nil? ? 0 : x[:metric].to_i) <=> (y[:metric].nil? ? 0 : y[:metric].to_i)
-    end.first
+    end
   end
 
   def interface_has_no_addresses_in_family?(iface, family)
@@ -447,7 +447,7 @@ Ohai.plugin(:Network) do
           iface[r[:dev]][:state] == "up" &&
           route_is_valid_default_route?(r, default_route)
       end
-    end.sort_by do |r|
+    end.min_by do |r|
       # sorting the selected routes:
       # - getting default routes first
       # - then sort by metric
@@ -463,7 +463,7 @@ Ohai.plugin(:Network) do
          0
        end,
       ]
-    end.first
+    end
   end
 
   # Both the network plugin and this plugin (linux/network) are run on linux. This plugin runs first.
@@ -496,18 +496,18 @@ Ohai.plugin(:Network) do
     if which("ip")
       # families to get default routes from
       families = [{
-                    :name => "inet",
-                    :default_route => "0.0.0.0/0",
-                    :default_prefix => :default,
-                    :neighbour_attribute => :arp,
+                    name: "inet",
+                    default_route: "0.0.0.0/0",
+                    default_prefix: :default,
+                    neighbour_attribute: :arp,
                   }]
 
       if ipv6_enabled?
         families << {
-                      :name => "inet6",
-                      :default_route => "::/0",
-                      :default_prefix => :default_inet6,
-                      :neighbour_attribute => :neighbour_inet6,
+                      name: "inet6",
+                      default_route: "::/0",
+                      default_prefix: :default_inet6,
+                      neighbour_attribute: :neighbour_inet6,
                     }
       end
 

data/lib/ohai/plugins/linux/platform.rb

@@ -24,8 +24,14 @@ Ohai.plugin(:Platform) do
     contents[/^Red Hat/i] ? "redhat" : contents[/(\w+)/i, 1].downcase
   end
 
+  # Amazon Linux AMI release 2013.09
+  # Amazon Linux 2
+  # Fedora release 28 (Twenty Eight)
+  # CentOS release 5.8 (Final)
+  # CentOS release 6.7 (Final)
+  # Red Hat Enterprise Linux Server release 7.5 (Maipo)
   def get_redhatish_version(contents)
-    contents[/Rawhide/i] ? contents[/((\d+) \(Rawhide\))/i, 1].downcase : contents[/release ([\d\.]+)/, 1]
+    contents[/Rawhide/i] ? contents[/((\d+) \(Rawhide\))/i, 1].downcase : contents[/(release)? ([\d\.]+)/, 2]
   end
 
   #

data/lib/ohai/plugins/netbsd/network.rb

@@ -71,7 +71,7 @@ Ohai.plugin(:Network) do
         if $4.empty?
           iface[cint][:addresses][$1] = { "family" => "inet6", "prefixlen" => $3 }
         else
-          #found a zone_id / scope
+          # found a zone_id / scope
           iface[cint][:addresses][$1] = { "family" => "inet6", "zoneid" => $2, "prefixlen" => $3, "scopeid" => $4 }
         end
       end

data/lib/ohai/plugins/network.rb

@@ -36,13 +36,13 @@ Ohai.plugin(:NetworkAddresses) do
     # grab ipaddress, scope, and iface for sorting later
     ipaddresses = []
     Mash[network["interfaces"]].each do |iface, iface_v|
-      next if iface_v.nil? || !iface_v.has_key?("addresses")
+      next if iface_v.nil? || !iface_v.key?("addresses")
       iface_v["addresses"].each do |addr, addr_v|
-        next if addr_v.nil? || (not addr_v.has_key? "family") || addr_v["family"] != family
+        next if addr_v.nil? || (not addr_v.key? "family") || addr_v["family"] != family
         ipaddresses << {
-          :ipaddress => addr_v["prefixlen"] ? IPAddress("#{addr}/#{addr_v["prefixlen"]}") : IPAddress("#{addr}/#{addr_v["netmask"]}"),
-          :scope => addr_v["scope"].nil? ? nil : addr_v["scope"].downcase,
-          :iface => iface,
+          ipaddress: addr_v["prefixlen"] ? IPAddress("#{addr}/#{addr_v["prefixlen"]}") : IPAddress("#{addr}/#{addr_v["netmask"]}"),
+          scope: addr_v["scope"].nil? ? nil : addr_v["scope"].downcase,
+          iface: iface,
         }
       end
     end

data/lib/ohai/plugins/openbsd/network.rb

@@ -71,7 +71,7 @@ Ohai.plugin(:Network) do
         if $4.empty?
           iface[cint][:addresses][$1] = { "family" => "inet6", "prefixlen" => $3 }
         else
-          #found a zone_id / scope
+          # found a zone_id / scope
           iface[cint][:addresses][$1] = { "family" => "inet6", "zoneid" => $2, "prefixlen" => $3, "scopeid" => $4 }
         end
       end

data/lib/ohai/plugins/packages.rb

@@ -27,7 +27,7 @@ Ohai.plugin(:Packages) do
       "DisplayVersion" => "version",
       "Publisher" => "publisher",
       "InstallDate" => "installdate",
-    }
+    }.freeze
   end
 
   collect_data(:linux) do

data/lib/ohai/plugins/passwd.rb

@@ -17,15 +17,15 @@ Ohai.plugin(:Passwd) do
       etc[:group] = Mash.new
 
       Etc.passwd do |entry|
-        user_passwd_entry = Mash.new(:dir => entry.dir, :gid => entry.gid, :uid => entry.uid, :shell => entry.shell, :gecos => entry.gecos)
+        user_passwd_entry = Mash.new(dir: entry.dir, gid: entry.gid, uid: entry.uid, shell: entry.shell, gecos: entry.gecos)
         user_passwd_entry.each_value { |v| fix_encoding(v) }
         entry_name = fix_encoding(entry.name)
-        etc[:passwd][entry_name] = user_passwd_entry unless etc[:passwd].has_key?(entry_name)
+        etc[:passwd][entry_name] = user_passwd_entry unless etc[:passwd].key?(entry_name)
       end
 
       Etc.group do |entry|
-        group_entry = Mash.new(:gid => entry.gid,
-                               :members => entry.mem.map { |u| fix_encoding(u) })
+        group_entry = Mash.new(gid: entry.gid,
+                               members: entry.mem.map { |u| fix_encoding(u) })
 
         etc[:group][fix_encoding(entry.name)] = group_entry
       end

data/lib/ohai/plugins/ruby.rb

@@ -30,19 +30,19 @@ Ohai.plugin(:Ruby) do
     languages[:ruby] = Mash.new
 
     values = {
-      :platform => "RUBY_PLATFORM",
-      :version => "RUBY_VERSION",
-      :release_date => "RUBY_RELEASE_DATE",
-      :target => "RbConfig::CONFIG['target']",
-      :target_cpu => "RbConfig::CONFIG['target_cpu']",
-      :target_vendor => "RbConfig::CONFIG['target_vendor']",
-      :target_os => "RbConfig::CONFIG['target_os']",
-      :host => "RbConfig::CONFIG['host']",
-      :host_cpu => "RbConfig::CONFIG['host_cpu']",
-      :host_os => "RbConfig::CONFIG['host_os']",
-      :host_vendor => "RbConfig::CONFIG['host_vendor']",
-      :bin_dir => "RbConfig::CONFIG['bindir']",
-      :ruby_bin => "::File.join(RbConfig::CONFIG['bindir'], RbConfig::CONFIG['ruby_install_name'])",
+      platform: "RUBY_PLATFORM",
+      version: "RUBY_VERSION",
+      release_date: "RUBY_RELEASE_DATE",
+      target: "RbConfig::CONFIG['target']",
+      target_cpu: "RbConfig::CONFIG['target_cpu']",
+      target_vendor: "RbConfig::CONFIG['target_vendor']",
+      target_os: "RbConfig::CONFIG['target_os']",
+      host: "RbConfig::CONFIG['host']",
+      host_cpu: "RbConfig::CONFIG['host_cpu']",
+      host_os: "RbConfig::CONFIG['host_os']",
+      host_vendor: "RbConfig::CONFIG['host_vendor']",
+      bin_dir: "RbConfig::CONFIG['bindir']",
+      ruby_bin: "::File.join(RbConfig::CONFIG['bindir'], RbConfig::CONFIG['ruby_install_name'])",
     }
 
     # Create a query string from above hash

data/lib/ohai/plugins/solaris2/dmi.rb

@@ -125,7 +125,7 @@ Ohai.plugin(:DMI) do
         dmi_record = {}
 
         # look up SMB ID
-        if smb_to_id.has_key?(header_information[3])
+        if smb_to_id.key?(header_information[3])
           id = smb_to_id[header_information[3]]
 
           # Don't overcapture for now (OHAI-260)
@@ -142,8 +142,8 @@ Ohai.plugin(:DMI) do
           next
         end
 
-        dmi[dmi_record[:type]] = Mash.new unless dmi.has_key?(dmi_record[:type])
-        dmi[dmi_record[:type]][:all_records] = [] unless dmi[dmi_record[:type]].has_key?(:all_records)
+        dmi[dmi_record[:type]] = Mash.new unless dmi.key?(dmi_record[:type])
+        dmi[dmi_record[:type]][:all_records] = [] unless dmi[dmi_record[:type]].key?(:all_records)
         dmi_record[:position] = dmi[dmi_record[:type]][:all_records].length
         dmi[dmi_record[:type]][:all_records].push(Mash.new)
         dmi[dmi_record[:type]][:all_records][dmi_record[:position]][:record_id] = header_information[1]

data/lib/ohai/plugins/solaris2/filesystem.rb

@@ -55,7 +55,7 @@ Ohai.plugin(:Filesystem) do
     so.stdout.lines do |line|
       next unless line =~ /^(.+?) on (.+?) (.+?) on (.+?)$/
       filesystem = $2
-      fs[filesystem] = Mash.new unless fs.has_key?(filesystem)
+      fs[filesystem] = Mash.new unless fs.key?(filesystem)
       fs[filesystem][:mount] = $1
       fs[filesystem][:mount_time] = $4 # $4 must come before "split", else it becomes nil
       fs[filesystem][:mount_options] = $3.split("/")
@@ -75,17 +75,17 @@ Ohai.plugin(:Filesystem) do
     so.stdout.lines do |line|
       next unless line =~ /^([^\t]+)\t([^\t]+)\t([^\t]+)\t([^\t]+)$/
       filesystem = $1
-      zfs[filesystem] = Mash.new unless zfs.has_key?(filesystem)
-      zfs[filesystem][:values] = Mash.new unless zfs[filesystem].has_key?("values")
-      zfs[filesystem][:sources] = Mash.new unless zfs[filesystem].has_key?("sources")
+      zfs[filesystem] = Mash.new unless zfs.key?(filesystem)
+      zfs[filesystem][:values] = Mash.new unless zfs[filesystem].key?("values")
+      zfs[filesystem][:sources] = Mash.new unless zfs[filesystem].key?("sources")
       zfs[filesystem][:values][$2] = $3
       zfs[filesystem][:sources][$2] = $4.chomp
     end
 
     zfs.each do |filesystem, attributes|
-      fs[filesystem] = Mash.new unless fs.has_key?(filesystem)
+      fs[filesystem] = Mash.new unless fs.key?(filesystem)
       fs[filesystem][:fs_type] = "zfs"
-      fs[filesystem][:mount] = attributes[:values][:mountpoint] if attributes[:values].has_key?("mountpoint")
+      fs[filesystem][:mount] = attributes[:values][:mountpoint] if attributes[:values].key?("mountpoint")
       fs[filesystem][:zfs_values] = attributes[:values]
       fs[filesystem][:zfs_sources] = attributes[:sources]
       # find all zfs parents