Merge pull request tubemogul#56 from ynnt/master

Add manage_user parameter

README.md

@@ -377,6 +377,11 @@ Path of the configuration file to be used by the aptly service.
 
 Default: `/etc/aptly.conf`
 
+##### `manage_user`
+Whethere should this module manage aptly user or not
+
+Default: `true`
+
 ##### `user`
 
 OS user which will run the service.

manifests/init.pp

@@ -14,6 +14,7 @@
   $port                 = $aptly::params::port,
   $bind                 = $aptly::params::bind,
   $config_filepath      = $aptly::params::config_filepath,
+  $manage_user          = $aptly::params::manage_user,
   $user                 = $aptly::params::user,
   $uid                  = $aptly::params::uid,
   $group                = $aptly::params::group,

manifests/install.pp

@@ -4,17 +4,15 @@
 #
 class aptly::install {
 
-  if ! defined(User[$aptly::user]) {
+  if $aptly::manage_user {
     user { $aptly::user:
       ensure  => present,
       uid     => $aptly::uid,
       gid     => $aptly::group,
       shell   => '/bin/bash',
       require => Group[$aptly::group],
     }
-  }
 
-  if ! defined(Group[$aptly::group]) {
     group { $aptly::group:
       ensure => present,
       gid    => $aptly::gid,

manifests/params.pp

@@ -15,6 +15,7 @@
   $port            = '8080'
   $bind            = '0.0.0.0'
   $config_filepath = '/etc/aptly.conf'
+  $manage_user     = true
   $user            = 'aptly'
   $uid             = 450
   $group           = 'aptly'