Move the $webhook::{user,group} definition to params.pp

Also, remove webhook::git_server as it seems useless
voxpupuli · May 24, 2015 · b0afdf4 · b0afdf4
1 parent e9885be
commit b0afdf4
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 6 deletions.
diff --git a/manifests/params.pp b/manifests/params.pp
@@ -28,8 +28,6 @@
   $mcollective = false
 
   # Webhook configuration information
-  $webhook_user                  = 'puppet'
-  $webhook_pass                  = 'puppet'
   $webhook_bind_address          = '0.0.0.0'
   $webhook_port                  = '8088'
   $webhook_access_logfile        = '/var/log/webhook/access.log'
@@ -71,13 +69,21 @@
     $modulepath      = "${r10k_basedir}/\$environment/modules:${pe_module_path}"
     $provider        = 'pe_gem'
     $r10k_binary     = 'r10k'
+
+    # webhook
+    $webhook_user    = 'peadmin'
+    $webhook_pass    = 'peadmin'
   } else {
     # Getting ready for FOSS support in this module
     $puppetconf_path = '/etc/puppet'
 
     # Mcollective configuration dynamic
     $modulepath = "${r10k_basedir}/\$environment/modules"
 
+    # webhook
+    $webhook_user    = 'puppet'
+    $webhook_pass    = 'puppet'
+
     case $::osfamily {
       'debian': {
         $plugins_dir     = '/usr/share/mcollective/plugins/mcollective'

diff --git a/manifests/webhook.pp b/manifests/webhook.pp
@@ -1,9 +1,8 @@
 # This class creates a github webhoook to allow curl style post-rec scripts
 class r10k::webhook(
-  $user  = 'peadmin',
-  $group = 'peadmin',
-  $git_server = 'localhost',
-  $webhook_bin_template = $::r10k::params::webhook_bin_template,
+  $user                 = $r10k::params::webhook_user,
+  $group                = $r10k::params::webhook_group,
+  $webhook_bin_template = $r10k::params::webhook_bin_template,
 ) inherits r10k::params {
 
   File {