From 7fa452d533e6765a925ae94d537174615a1b0fa5 Mon Sep 17 00:00:00 2001 From: Dan Prince Date: Mon, 4 Mar 2013 15:34:36 -0500 Subject: [PATCH] Don't require /etc/init.d/snmpd on RHEL. Updates the snmpd module so that we don't require /etc/init.d/snmpd to start the snmpd service. This should fix issues running this module on RHEL. On Ubuntu we explicitly create /etc/init.d/snmpd and already has some resource chaining in place so that it will conditionally run before the snpmd service starts. Change-Id: I480c38d26c8b423f5a8d1ddc405749c03aae19b1 Reviewed-on: https://review.openstack.org/23460 Reviewed-by: Clark Boylan Approved: Jeremy Stanley Reviewed-by: Jeremy Stanley Tested-by: Jenkins --- modules/snmpd/manifests/init.pp | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/modules/snmpd/manifests/init.pp b/modules/snmpd/manifests/init.pp index e4fa2b3759..e7f0c509d0 100644 --- a/modules/snmpd/manifests/init.pp +++ b/modules/snmpd/manifests/init.pp @@ -11,10 +11,7 @@ class snmpd { service { 'snmpd': ensure => running, hasrestart => true, - require => [ - File['/etc/snmp/snmpd.conf'], - File['/etc/init.d/snmpd'], - ], + require => File['/etc/snmp/snmpd.conf'] } if ($::operatingsystem == 'Ubuntu') {