switch to parameterized classes, changing the variable names as appropriate to remove...
authorMicah Anderson <micah@riseup.net>
Wed, 3 Apr 2013 17:18:26 +0000 (13:18 -0400)
committerMicah Anderson <micah@riseup.net>
Wed, 3 Apr 2013 17:18:26 +0000 (13:18 -0400)
manifests/debian.pp
manifests/init.pp
manifests/linux.pp
templates/Debian/default

index a480a2c..156ef75 100644 (file)
@@ -11,16 +11,16 @@ class stunnel::debian inherits stunnel::linux {
 
   # make the /etc/default/stunnel ENABLED configurable with a variable
   # and default to on
-  case $stunnel_startboot {
-    '': { $stunnel_startboot = '1' }
-    default: { $stunnel_startboot = '1' }
+  case $startboot {
+    '': { $startboot = '1' }
+    default: { $startboot = '1' }
   }
 
   # make the /etc/default/stunnel extra configurable with a variable
   # and default to adding nothing to the default file
-  case $stunnel_default_extra {
-    '': { $stunnel_default_extra = '' }
-    default: { $stunnel_default_extra = '' }
+  case $default_extra {
+    '': { $default_extra = '' }
+    default: { $default_extra = '' }
   }
 
   file { '/etc/default/stunnel4':
index b1437d2..270540d 100644 (file)
 
 # TODO: warn on cert/key issues, fail on false accept?
 
-class stunnel {
-
-  case $stunnel_ensure_version {
-    '': { $stunnel_ensure_version = 'present' }
-    default: { $stunnel_ensure_version = 'present' }
-  }
-
+class stunnel ( $ensure_version = 'present', $startboot = '1', $default_extra )
+{
   case $::operatingsystem {
-    debian: { include stunnel::debian }
-    centos: { include stunnel::centos }
-    default: { include stunnel::default }
+    debian: { class { 'stunnel::debian': } }
+    centos: { class {  'stunnel::centos': } }
+    default: { class { 'stunnel::default': } }
   }
 
   if $use_nagios {
index b4b99c6..eb1b10a 100644 (file)
@@ -1,7 +1,7 @@
 class stunnel::linux inherits stunnel::base {
 
-  if $stunnel_ensure_version == '' { $stunnel_ensure_version = 'installed' }
+  if $ensure_version == '' { $ensure_version = 'installed' }
   package { 'stunnel':
-    ensure => $stunnel_ensure_version
+    ensure => $ensure_version
   }
 }
index 85c4754..ccfefa5 100644 (file)
@@ -3,11 +3,11 @@
 # September 2003
 
 # Change to one to enable stunnel automatic startup
-ENABLED=<%= stunnel_startboot %>
+ENABLED=<%= startboot %>
 FILES="/etc/stunnel/*.conf"
 OPTIONS=""
 
 # Change to one to enable ppp restart scripts
 PPP_RESTART=0
 
-<%= stunnel_default_extra %>
+<%= default_extra %>