Re: [Puppet-dev] [PATCH/puppet 1/1] (#6324) Always fall back to svcadm enable except for the maintenance state

2011-02-18 Thread Dominic Cleal
On 15/02/11 21:21, Luke Kanies wrote: > Good catch, and definitely a good idea. > > I don't suppose you feel like creating a set of SMF tests for this? :) I've given it a go, but getting the rspec to work was a steep learning curve! I'll resend the patches. -- Dominic Cleal Red Hat Consulting

Re: [Puppet-dev] [PATCH/puppet 1/1] (#6324) Always fall back to svcadm enable except for the maintenance state

2011-02-15 Thread Luke Kanies
Good catch, and definitely a good idea. I don't suppose you feel like creating a set of SMF tests for this? :) On Feb 15, 2011, at 4:06 AM, Dominic Cleal wrote: > If state is running, using svcadm enable is harmless and prevents errors with > execute(). > > Signed-off-by: Dominic Cleal > --- >

[Puppet-dev] [PATCH/puppet 1/1] (#6324) Always fall back to svcadm enable except for the maintenance state

2011-02-15 Thread Dominic Cleal
If state is running, using svcadm enable is harmless and prevents errors with execute(). Signed-off-by: Dominic Cleal --- Local-branch: tickets/master/6324 lib/puppet/provider/service/smf.rb |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/puppet/provider/service/s