Merge pull request #222 from jonny0303/jonny0303-patch-1

Don't execute the lvm commands when not supported. #193

Author: tphoney

lib/facter/lvm_support.rb

@@ -15,8 +15,10 @@
 Facter.add('lvm_vgs') do
   confine :lvm_support => true
 
-  vgs = Facter::Core::Execution.execute('vgs -o name --noheadings 2>/dev/null', timeout: 30)
-
+  if Facter.value(:lvm_support)
+    vgs = Facter::Core::Execution.execute('vgs -o name --noheadings 2>/dev/null', timeout: 30)
+  end
+  
   if vgs.nil?
     setcode { 0 }
   else
@@ -46,8 +48,9 @@
 pv_list = []
 Facter.add('lvm_pvs') do
   confine :lvm_support => true
-
-  pvs = Facter::Core::Execution.execute('pvs -o name --noheadings 2>/dev/null', timeout: 30)
+  if Facter.value(:lvm_support)
+    pvs = Facter::Core::Execution.execute('pvs -o name --noheadings 2>/dev/null', timeout: 30)
+  end
 
   if pvs.nil?
     setcode { 0 }