diff options
author | James Turnbull <james@lovedthanlost.net> | 2008-07-17 11:12:35 +1000 |
---|---|---|
committer | James Turnbull <james@lovedthanlost.net> | 2008-07-17 11:13:01 +1000 |
commit | de6aec6e37edb9c575f9223686da03cea4063ad7 (patch) | |
tree | f9a6fa2efb700e3fe64a3186cb901973f36599ea /lib/puppet/provider | |
parent | 0a0fcafa3e2405aa18ecc62d1e6dc69bbd9883a1 (diff) | |
download | puppet-de6aec6e37edb9c575f9223686da03cea4063ad7.tar.gz puppet-de6aec6e37edb9c575f9223686da03cea4063ad7.tar.xz puppet-de6aec6e37edb9c575f9223686da03cea4063ad7.zip |
Fix Ticket 1426 - services on redhat are restarted again
as far I see there have been a regression while refactoring the redhat service provider to user /sbin/service. This commit fixes this bug (1426) and service restarts are working again on redhat based systems.
There are no tests, as I couldn't figure out how that should be tested. It seems that some restart logic is already tested, however it looks like not every single kind of provider is covered by tests, nor I see at the moment how I could do that.
Diffstat (limited to 'lib/puppet/provider')
-rwxr-xr-x | lib/puppet/provider/service/redhat.rb | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/lib/puppet/provider/service/redhat.rb b/lib/puppet/provider/service/redhat.rb index e2d6ac947..48da577ec 100755 --- a/lib/puppet/provider/service/redhat.rb +++ b/lib/puppet/provider/service/redhat.rb @@ -50,10 +50,18 @@ Puppet::Type.type(:service).provide :redhat, :parent => :init do end def restart - if @resource[:hasrestart] == true + if @resource[:hasrestart] == :true service(@resource[:name], "restart") else - return false + super + end + end + + def status + if @resource[:hasstatus] == :true + service(@resource[:name], "status") + else + super end end |