Issue #7165 has been updated by Jo Rhett.

It would appear that the service resource isn't checked to see if its running 
until the notify is parsed, which is why the ordering happened as listed.  You 
could enforce this ordering by adding "require => Service['nginx']" to the file 
stanza, which would create

start service
replace file
restart service

What I think you're asking for is "if the service hasn't started yet, update 
the file first and then start the service".  I'm trying to think of a reason 
this couldn't be the default behavior in case of a Notify to an unstarted 
service and I can't think of one.  If there are examples of when this would not 
be desirable then, then it would need explicit syntax.
(not speaking for anyone besides myself)
----------------------------------------
Bug #7165: services restarted unnecessarily
https://projects.puppetlabs.com/issues/7165

Author: John Florian
Status: Needs Decision
Priority: Normal
Assignee: Nigel Kersten
Category: service
Target version: 
Affected Puppet version: 0.25.5
Keywords: 
Branch: 


Given the following conditions:

1. service 'S' is currently disabled/stopped
1. config file 'F' for service 'S' is currently in state 'A'
1. puppet class 'C' calls for 'S' to be enabled/running and 'F' to be in state 
'B'

When puppet runs, the following currently occurs:
<pre>
notice: //C/File[F]/content: content changed '{md5}-A' to '{md5}-B'
info: //C/File[F]: Scheduling refresh of Service[S]
notice: //C/Service[S]/ensure: ensure changed 'stopped' to 'running'
notice: //C/Service[S]: Triggering 'refresh' from 1 dependencies
</pre>

Thus, puppet sees the service is to be running and makes it so.  Then it 
realizes that a refresh is needed and so restarts the service.  The restart is 
unnecessary in this case.  Depending on the particular service, this could be 
quite undesirable.  Puppet should be able to recognize this case and simply 
start the service just once.



-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to