Facts are now core to Puppet 3.8 and not needed in this module

[mbentle8]

Nan,

Could you remove the 4 custom facts from this module and make a tagged release that we could use without those? (ldom, zfs_version, zonename, zpool_version)

These are now core facts in PE3.8/facter 2.4, and the ldom.rb had a bug affecting us on solaris 11 x86 that has been fixed since facter 2.3, but since we're also using this module, your version of ldom.rb is overriding the version that comes with facter. The root of the problem is that the 'virtinfo' command doesn't exist on Solaris 11.1 x86, and facter bails out with an error, causing puppet runs to fail. Again, this was fixed in facter 2.3 (See: https://tickets.puppetlabs.com/browse/FACT-654 )