Merge pull request redhat-openstack#223 from sijis/add_packages

Add redhat packages on plugins

manifests/plugin/amqp.pp

@@ -16,6 +16,12 @@
 
   validate_bool($amqppersistent)
 
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-amqp':
+      ensure => $ensure,
+    }
+  }
+
   collectd::plugin {'amqp':
     ensure   => $ensure,
     content  => template('collectd/plugin/amqp.conf.erb'),

manifests/plugin/bind.pp

@@ -24,6 +24,12 @@
   )
   validate_array($views)
 
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-bind':
+      ensure => $ensure,
+    }
+  }
+
   collectd::plugin {'bind':
     ensure   => $ensure,
     content  => template('collectd/plugin/bind.conf.erb'),

manifests/plugin/curl.pp

@@ -4,6 +4,13 @@
   $interval = undef,
   $pages    = { },
 ) {
+
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-curl':
+      ensure => $ensure,
+    }
+  }
+
   collectd::plugin {'curl':
     ensure   => $ensure,
     interval => $interval,

manifests/plugin/curl_json.pp

@@ -11,6 +11,12 @@
   include collectd::params
   validate_hash($keys)
 
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-curl_json':
+      ensure => $ensure,
+    }
+  }
+
   $conf_dir = $collectd::params::plugin_conf_dir
 
   # This is deprecated file naming ensuring old style file removed, and should be removed in next major relese

manifests/plugin/iptables.pp

@@ -6,6 +6,12 @@
 ) {
   validate_hash($chains)
 
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-iptables':
+      ensure => $ensure,
+    }
+  }
+
   collectd::plugin {'iptables':
     ensure   => $ensure,
     content  => template('collectd/plugin/iptables.conf.erb'),

manifests/plugin/mysql.pp

@@ -3,6 +3,13 @@
 class collectd::plugin::mysql (
   $interval = undef,
 ){
+
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-mysql':
+      ensure => $ensure,
+    }
+  }
+
   collectd::plugin { 'mysql':
     interval => $interval,
   }

manifests/plugin/nginx.pp

@@ -10,6 +10,12 @@
   $interval   = undef,
 ) {
 
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-nginx':
+      ensure => $ensure,
+    }
+  }
+
   collectd::plugin {'nginx':
     ensure   => $ensure,
     content  => template('collectd/plugin/nginx.conf.erb'),

manifests/plugin/perl.pp

@@ -8,6 +8,12 @@
   include collectd::params
   $conf_dir = $collectd::params::plugin_conf_dir
 
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-perl':
+      ensure => $ensure,
+    }
+  }
+
   collectd::plugin { 'perl':
     ensure   => $ensure,
     globals  => true,

manifests/plugin/ping.pp

@@ -12,6 +12,12 @@
 
   validate_array($hosts)
 
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-ping':
+      ensure => $ensure,
+    }
+  }
+
   $conf_dir = $collectd::params::plugin_conf_dir
 
   file {

manifests/plugin/postgresql.pp

@@ -8,6 +8,12 @@
 ) {
   include collectd::params
 
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-postgresql':
+      ensure => $ensure,
+    }
+  }
+
   collectd::plugin {'postgresql':
     ensure   => $ensure,
     interval => $interval,

manifests/plugin/python.pp

@@ -11,6 +11,12 @@
 
   validate_hash($config)
 
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-python':
+      ensure => $ensure,
+    }
+  }
+
   $conf_dir = $collectd::params::plugin_conf_dir
 
   # This is deprecated file naming ensuring old style file removed, and should be removed in next major relese

manifests/plugin/sensors.pp

@@ -7,6 +7,12 @@
   $interval         = undef,
 ) {
 
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-sensors':
+      ensure => $ensure,
+    }
+  }
+
   collectd::plugin {'sensors':
     ensure   => $ensure,
     content  => template('collectd/plugin/sensors.conf.erb'),

manifests/plugin/snmp.pp

@@ -7,6 +7,12 @@
 ) {
   validate_hash($data, $hosts)
 
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-snmp':
+      ensure => $ensure,
+    }
+  }
+
   collectd::plugin {'snmp':
     ensure   => $ensure,
     content  => template('collectd/plugin/snmp.conf.erb'),

manifests/plugin/varnish.pp

@@ -13,7 +13,7 @@
 
   if $::osfamily == 'Redhat' {
     package { 'collectd-varnish':
-      ensure => installed
+      ensure => $ensure,
     }
   }
 

manifests/plugin/write_http.pp

@@ -7,6 +7,12 @@
 
   validate_hash($urls)
 
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-write_http':
+      ensure => $ensure,
+    }
+  }
+
   collectd::plugin {'write_http':
     ensure   => $ensure,
     content  => template('collectd/plugin/write_http.conf.erb'),

manifests/plugin/write_riemann.pp

@@ -11,6 +11,12 @@
   validate_bool($store_rates)
   validate_bool($always_append_ds)
 
+  if $::osfamily == 'Redhat' {
+    package { 'collectd-write_riemann':
+      ensure => $ensure,
+    }
+  }
+
   collectd::plugin {'write_riemann':
     ensure   => $ensure,
     content  => template('collectd/plugin/write_riemann.conf.erb'),