RHEL7 does not need the collectd-python package about puppet-collectd HOT 7 CLOSED

--- a/modules/collectd/manifests/plugin/python.pp
+++ b/modules/collectd/manifests/plugin/python.pp
@@ -12,9 +12,9 @@ define collectd::plugin::python (
   validate_hash($config)

   if $::osfamily == 'Redhat' {
-    package { 'collectd-python':
-      ensure => $ensure,
-    }
+#    package { 'collectd-python':
+#      ensure => $ensure,
+#    }
   }

from puppet-collectd.

Thanks for the contribution. Can you open this as a pull request?

from puppet-collectd.

Also, can you remove the package resource rather than commenting it out?

from puppet-collectd.

You need to add a conditional on the version of redhat around the package resource. You can't just comment it out or it breaks the module for RHEL6 and RHEL5.

from puppet-collectd.

This wasn't actually meant as a PR, put as a tip to where to change the code :)

from puppet-collectd.

Using ensure_packages from stdlib would be very appreciated....

from puppet-collectd.

Released in #254

from puppet-collectd.