opendev/system-config
Code Issues Proposed changes

Don't require /etc/init.d/snmpd on RHEL.

Browse Source 
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 <clark.boylan@gmail.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
This commit is contained in:
Dan Prince 2013-03-04 15:34:36 -05:00 committed by Jenkins
parent b8764f9551
commit 7fa452d533
1 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
modules/snmpd/manifests/init.pp
View File

@ -11,10 +11,7 @@ class snmpd {
service { 'snmpd': service { 'snmpd':
ensure => running, ensure => running,
hasrestart => true, hasrestart => true,
require => [ require => File['/etc/snmp/snmpd.conf']
File['/etc/snmp/snmpd.conf'],
File['/etc/init.d/snmpd'],
],
} }
if ($::operatingsystem == 'Ubuntu') { if ($::operatingsystem == 'Ubuntu') {