Merge pull request #735 from bulletproofnetworks/master

wilson208 · web-flow · commit 70d071ba6fa3 · 2017-03-10T10:25:56.000Z
Should only try to apply the resource if it not defined
diff --git a/lib/puppet/parser/functions/ensure_packages.rb b/lib/puppet/parser/functions/ensure_packages.rb
@@ -37,7 +37,9 @@ module Puppet::Parser::Functions
 
       Puppet::Parser::Functions.function(:ensure_resource)
       packages.each { |package_name|
+      if !findresource("Package[#{package_name}]")
         function_ensure_resource(['package', package_name, defaults ])
+      end
     }
     end
   end

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,9 @@ module Puppet::Parser::Functions`
`37`	`37`
`38`	`38`	`Puppet::Parser::Functions.function(:ensure_resource)`
`39`	`39`	`packages.each { \|package_name\|`
	`40`	`+ if !findresource("Package[#{package_name}]")`
`40`	`41`	`function_ensure_resource(['package', package_name, defaults ])`
	`42`	`+ end`
`41`	`43`	`}`
`42`	`44`	`end`
`43`	`45`	`end`