changed a bit API of puppet class after rolling out the setup to production

Author: Roman Plessl

README.md

@@ -25,11 +25,11 @@ These configuration parameter can be set:
   $reporting_disabled             = false
 
   # [meta]
-  $hostname                       = 'localhost' or 192.168.0.1'
-  $peers                          = 'none'      or ['192.168.0.2', '192.168.0.3']
+  $meta_hostname                  = "localhost" or "192.168.0.1"
+  $meta_peers                     = undef       or ["192.168.0.1:8088", "192.168.0.2:8088", "192.168.0.3:8088"]
 
   # [retention]
-  $replication                    = 'none'      or 3
+  $retention_replication          = undef       or 3
 ```
 
 ## Testing

manifests/config.pp

@@ -24,23 +24,23 @@
   ini_setting { 'meta_hostname':
     section => 'meta',
     setting => 'hostname',
-    value   => "\"${influxdb::hostname}\"",
+    value   => "\"${influxdb::meta_hostname}\"",
   }
 
-  if $influxdb::peers != 'none' {
+  if $influxdb::meta_peers != undef {
     ini_setting { 'meta_peers':
       section => 'meta',
       setting => 'peers',
-      value   => '\"${influxdb::peers}\"',
+      value   => $influxdb::meta_peers,
     }
   }
 
-  if $influxdb::replication != 'none' {
+  if $influxdb::retention_replication != undef {
     # [retention]
     ini_setting { 'retention_replication':
       section => 'retention',
       setting => 'replication',
-      value   => $influxdb::replication,
+      value   => $influxdb::retention_replication,
     }
   }
 

manifests/init.pp

@@ -5,9 +5,9 @@
   $install_from_repository  = $influxdb::params::install_from_repository,
   $config_file              = $influxdb::params::config_file,
   $reporting_disabled       = $influxdb::params::reporting_disabled,
-  $hostname                 = $influxdb::params::hostname,
-  $peers                    = $influxdb::params::peers,
-  $replication              = $influxdb::params::replication,
+  $meta_hostname            = $influxdb::params::meta_hostname,
+  $meta_peers               = $influxdb::params::meta_peers,
+  $retention_replication    = $influxdb::params::retention_replication,
 ) inherits influxdb::params {
 
   class { 'influxdb::install': } ->

manifests/params.pp

@@ -1,18 +1,18 @@
 # == Class: influxdb::params
 # DO NOT CALL DIRECTLY
 class influxdb::params {
-  $ensure                               = 'installed'
-  $version                              = '0.9.1'
-  $install_from_repository              = true
-  $config_file                          = '/etc/opt/influxdb/influxdb.conf'
+  $ensure                  = 'installed'
+  $version                 = '0.9.1'
+  $install_from_repository = true
+  $config_file             = '/etc/opt/influxdb/influxdb.conf'
 
-  # general section of config.toml
-  $reporting_disabled                   = false
+  # general section of influxdb.conf
+  $reporting_disabled      = false
 
   # [meta]
-  $hostname                             = 'localhost'
-  $peers                                = 'none'
+  $meta_hostname           = 'localhost'
+  $meta_peers              = undef
 
   # [retention]
-  $replication                          = 'none'
+  $retention_replication   = undef
 }