Changes package installation

* Right now package is checked with a defined function * If user doesn’t want to manage the package or is managing somewhere else, they should use the parameter not to manage it
voxpupuli · Oct 14, 2016 · 585f00c · 585f00c
1 parent 9425766
commit 585f00c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/manifests/install.pp b/manifests/install.pp
@@ -3,10 +3,11 @@
 # This class installs the application.
 #
 class redis::install {
-  unless defined(Package['$::redis::package_name']) {
-    ensure_resource('package', $::redis::package_name, {
-      'ensure' => $::redis::package_ensure
-    })
+
+  if $::redis::manage_package {
+    package { $::redis::package_name:
+      ensure => $::redis::package_ensure,
+    }
   }
 }
 
diff --git a/manifests/params.pp b/manifests/params.pp
@@ -5,6 +5,7 @@
 class redis::params {
   # Generic
   $manage_repo = false
+  $manage_package = true
 
   # redis.conf.erb
   $activerehashing                 = true