From a403c09e75ed7e50497b6ccf467dc50c63244942 Mon Sep 17 00:00:00 2001 From: Steffen Zieger Date: Fri, 5 Sep 2014 15:20:08 +0200 Subject: fix some lint warnings, fixes #100 --- manifests/init.pp | 12 +++++++----- manifests/modload.pp | 24 +++++++++++------------- manifests/params.pp | 3 ++- manifests/snippet.pp | 3 ++- tests/log_templates.pp | 4 ++-- tests/multiple_hosts.pp | 18 +++++++++--------- 6 files changed, 33 insertions(+), 31 deletions(-) diff --git a/manifests/init.pp b/manifests/init.pp index aad06d5..5115674 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -1,6 +1,7 @@ # == Class: rsyslog # -# Meta class to install rsyslog with a basic configuration. You probably want rsyslog::client or rsyslog::server +# Meta class to install rsyslog with a basic configuration. +# You probably want rsyslog::client or rsyslog::server # # === Parameters # @@ -39,13 +40,14 @@ class rsyslog ( $modules = $rsyslog::params::modules, $preserve_fqdn = $rsyslog::params::preserve_fqdn, $max_message_size = $rsyslog::params::max_message_size, - $extra_modules = $rsyslog::params::extra_modules + $extra_modules = $rsyslog::params::extra_modules ) inherits rsyslog::params { class { 'rsyslog::install': } class { 'rsyslog::config': } + if $extra_modules != [] { - class { 'rsyslog::modload': } - } - class { 'rsyslog::service': } + class { 'rsyslog::modload': } + } + class { 'rsyslog::service': } } diff --git a/manifests/modload.pp b/manifests/modload.pp index 8022cf0..7a838af 100644 --- a/manifests/modload.pp +++ b/manifests/modload.pp @@ -2,16 +2,14 @@ # class rsyslog::modload ( - $modload_filename = '10-modload.conf', -) - { - file { "${rsyslog::rsyslog_d}$modload_filename": - ensure => file, - owner => 'root', - group => $rsyslog::run_group, - content => template('rsyslog/modload.erb'), - require => Class['rsyslog::install'], - notify => Class['rsyslog::service'], - } - } - + $modload_filename = '10-modload.conf', +) { + file { "${rsyslog::rsyslog_d}${modload_filename}": + ensure => file, + owner => 'root', + group => $rsyslog::run_group, + content => template('rsyslog/modload.erb'), + require => Class['rsyslog::install'], + notify => Class['rsyslog::service'], + } +} diff --git a/manifests/params.pp b/manifests/params.pp index 0a03d07..9027836 100644 --- a/manifests/params.pp +++ b/manifests/params.pp @@ -1,6 +1,7 @@ # == Class: rsyslog::params # -# This defines default configuration values for rsyslog. You don't want to use it directly. +# This defines default configuration values for rsyslog. +# You don't want to use it directly. # # === Parameters # diff --git a/manifests/snippet.pp b/manifests/snippet.pp index 9edf179..f638396 100644 --- a/manifests/snippet.pp +++ b/manifests/snippet.pp @@ -1,6 +1,7 @@ # == Define: rsyslog::snippet # -# This class allows for you to create a rsyslog configuration file with whatever content you pass in. +# This class allows for you to create a rsyslog configuration file with +# whatever content you pass in. # # === Parameters # diff --git a/tests/log_templates.pp b/tests/log_templates.pp index 79f1e8f..a6bf75b 100644 --- a/tests/log_templates.pp +++ b/tests/log_templates.pp @@ -1,4 +1,4 @@ -class{'rsyslog::client': +class { 'rsyslog::client': log_templates => [ { name => 'RFC3164fmt', @@ -6,4 +6,4 @@ class{'rsyslog::client': }, ], actionfiletemplate => 'RFC3164fmt', -} \ No newline at end of file +} diff --git a/tests/multiple_hosts.pp b/tests/multiple_hosts.pp index 0852047..9e5a60e 100644 --- a/tests/multiple_hosts.pp +++ b/tests/multiple_hosts.pp @@ -1,17 +1,17 @@ -class{'rsyslog::client': +class { 'rsyslog::client': remote_servers => [ { - host => 'logs.example.org', + host => 'logs.example.org', }, { - port => '55514', + port => '55514', }, { - host => 'logs.somewhere.com', - port => '555', - pattern => '*.log', - protocol => 'tcp', - format => 'RFC3164fmt', + host => 'logs.somewhere.com', + port => '555', + pattern => '*.log', + protocol => 'tcp', + format => 'RFC3164fmt', }, ] -} \ No newline at end of file +} -- cgit v1.2.3