]> review.fuel-infra Code Review - puppet-modules/puppetlabs-apt.git/commitdiff
Merge pull request #85 from dalen/pin_explanation
authorBranan Purvine-Riley <branan@puppetlabs.com>
Mon, 29 Oct 2012 15:31:34 +0000 (08:31 -0700)
committerBranan Purvine-Riley <branan@puppetlabs.com>
Mon, 29 Oct 2012 15:31:34 +0000 (08:31 -0700)
(#16076) Ability to fill in pin explanation

manifests/pin.pp
spec/defines/pin_spec.rb
templates/pin.pref.erb

index cebc6e7b0da2c390092324ad7c32c5e236dab641..21cc3ffdefc0294c0c91c37de474971b94b26869 100644 (file)
@@ -3,6 +3,7 @@
 
 define apt::pin(
   $ensure     = present,
+  $explanation = "${::caller_module_name}: ${name}",
   $order      = '',
   $packages   = '*',
   $priority   = 0,
index cd269d3b8bb11b3deff3831df99be63b5d877da8..3aaf49cef494e79f3b4fce0c2e2e02ba91d7915d 100644 (file)
@@ -50,7 +50,7 @@ describe 'apt::pin', :type => :define do
           'owner'   => 'root',
           'group'   => 'root',
           'mode'    => '0644',
-          'content' => "# #{title}\nPackage: #{param_hash[:packages]}\nPin: release a=#{param_hash[:release] || title}\nPin-Priority: #{param_hash[:priority]}\n",
+          'content' => "# #{title}\nExplanation: : #{title}\nPackage: #{param_hash[:packages]}\nPin: release a=#{param_hash[:release] || title}\nPin-Priority: #{param_hash[:priority]}\n",
         })
       }
     end
index 9a1e690ee66b49254e14f63aa265396460ec6d53..74b92659b87352db11945d7e1cd07971c1ec2613 100644 (file)
@@ -1,4 +1,5 @@
 # <%= name %>
+Explanation: <%= explanation %>
 Package: <%= packages %>
 Pin: <%= pin %>
 Pin-Priority: <%= priority %>