On Monday, May 7, 2012 8:06:22 AM UTC-7, Nan Liu wrote: > > First off, thanks for taking the time to write and share your smf > property provider. I've list of some comments/suggestions below. > > ... > > > > svccfg("-s",resource[:fmri],"setprop",resource[:property],"=",resource[:value]) > > > > X refresh="/usr/sbin/svcadm refresh #{@resource[:fmri]}" > > X execute(refresh) > > Any reason not using? > > svcadm('refresh', @resource[:frmri]) >
oops. > You can add defaultto :present in ensurable to avoid requiring users > specify ensure => present. > > Thanks for the tips, Nan -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To view this discussion on the web visit https://groups.google.com/d/msg/puppet-dev/-/r-s38gb-SG0J. To post to this group, send email to puppet-dev@googlegroups.com. To unsubscribe from this group, send email to puppet-dev+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.