X-Git-Url: https://review.fuel-infra.org/gitweb?a=blobdiff_plain;f=lib%2Ffacter%2Fapt_updates.rb;h=cf7d12f4c2d2e2593d95a9e93f45a01e0314fde6;hb=ab5c977b5d66840a74df12a7e64d7dff144acd73;hp=e880fdf29841b264b01335b709caf30f78693d15;hpb=ebf9d9f6a629dc69973cad6092080f36ec5af1d6;p=puppet-modules%2Fpuppetlabs-apt.git

diff --git a/lib/facter/apt_updates.rb b/lib/facter/apt_updates.rb
index e880fdf..cf7d12f 100644
--- a/lib/facter/apt_updates.rb
+++ b/lib/facter/apt_updates.rb
@@ -1,38 +1,49 @@
 apt_package_updates = nil
-Facter.add("apt_has_updates") do
-  confine :osfamily => 'Debian'
-  if File.executable?("/usr/bin/apt-get")
-    apt_get_result = Facter::Util::Resolution.exec('/usr/bin/apt-get -s upgrade 2>&1')
-    if not apt_get_result.nil?
-      apt_package_updates = [[], []]
+apt_dist_updates = nil
+
+def get_updates(upgrade_option)
+  apt_updates = nil
+  if File.executable?('/usr/bin/apt-get')
+    apt_get_result = Facter::Util::Resolution.exec("/usr/bin/apt-get -s -o Debug::NoLocking=true #{upgrade_option} 2>&1")
+    unless apt_get_result.nil?
+      apt_updates = [[], []]
       apt_get_result.each_line do |line|
-        if line =~ /^Inst\s/
-          package = line.gsub(/^Inst\s([^\s]+)\s.*/, '\1').strip
-          apt_package_updates[0].push(package)
-          security_matches = [
-            / Debian[^\s]+-updates /,
-            / Debian-Security:/,
-            / Ubuntu[^\s]+-security /,
-            / gNewSense[^\s]+-security /
-          ]
-          re = Regexp.union(security_matches)
-          if line.match(re)
-            apt_package_updates[1].push(package)
-          end
+        next unless line =~ %r{^Inst\s}
+        package = line.gsub(%r{^Inst\s([^\s]+)\s.*}, '\1').strip
+        apt_updates[0].push(package)
+        security_matches = [
+          %r{ Debian-Security:},
+          %r{ Ubuntu[^\s]+-security[, ]},
+          %r{ gNewSense[^\s]+-security[, ]},
+        ]
+        re = Regexp.union(security_matches)
+        if line.match(re)
+          apt_updates[1].push(package)
         end
       end
     end
   end
 
   setcode do
-    if not apt_package_updates.nil? and apt_package_updates.length == 2
-      apt_package_updates != [[], []]
+    if !apt_updates.nil? && apt_updates.length == 2
+      apt_updates != [[], []]
     end
   end
+  apt_updates
+end
+
+Facter.add('apt_has_updates') do
+  confine osfamily: 'Debian'
+  apt_package_updates = get_updates('upgrade')
+end
+
+Facter.add('apt_has_dist_updates') do
+  confine osfamily: 'Debian'
+  apt_dist_updates = get_updates('dist-upgrade')
 end
 
-Facter.add("apt_package_updates") do
-  confine :apt_has_updates => true
+Facter.add('apt_package_updates') do
+  confine apt_has_updates: true
   setcode do
     if Facter.version < '2.0.0'
       apt_package_updates[0].join(',')
@@ -42,16 +53,63 @@ Facter.add("apt_package_updates") do
   end
 end
 
-Facter.add("apt_updates") do
-  confine :apt_has_updates => true
+Facter.add('apt_package_dist_updates') do
+  confine apt_has_dist_updates: true
+  setcode do
+    if Facter.version < '2.0.0'
+      apt_dist_updates[0].join(',')
+    else
+      apt_dist_updates[0]
+    end
+  end
+end
+
+Facter.add('apt_package_security_updates') do
+  confine apt_has_updates: true
+  setcode do
+    if Facter.version < '2.0.0'
+      apt_package_updates[1].join(',')
+    else
+      apt_package_updates[1]
+    end
+  end
+end
+
+Facter.add('apt_package_security_dist_updates') do
+  confine apt_has_dist_updates: true
+  setcode do
+    if Facter.version < '2.0.0'
+      apt_dist_updates[1].join(',')
+    else
+      apt_dist_updates[1]
+    end
+  end
+end
+
+Facter.add('apt_updates') do
+  confine apt_has_updates: true
   setcode do
     Integer(apt_package_updates[0].length)
   end
 end
 
-Facter.add("apt_security_updates") do
-  confine :apt_has_updates => true
+Facter.add('apt_dist_updates') do
+  confine apt_has_dist_updates: true
+  setcode do
+    Integer(apt_dist_updates[0].length)
+  end
+end
+
+Facter.add('apt_security_updates') do
+  confine apt_has_updates: true
   setcode do
     Integer(apt_package_updates[1].length)
   end
 end
+
+Facter.add('apt_security_dist_updates') do
+  confine apt_has_dist_updates: true
+  setcode do
+    Integer(apt_dist_updates[1].length)
+  end
+end