Removes Exec['reload-nginx'], because Service['nginx'] can also do the thing. #9

Merged
merged 1 commit into from Feb 29, 2012

Conversation

Projects
None yet
2 participants
Contributor

LarsFronius commented Feb 27, 2012

I just think that it is more intuitive to write notify Service, than using an additionally defined Exec thingy. It is also on the road-map to really support Service with refresh option (http://projects.puppetlabs.com/issues/1014) right now, you can just set "restart => '/etc/init.d/nginx reload'" and it does what you expect.

BenoitCattie added a commit that referenced this pull request Feb 29, 2012

Merge pull request #9 from LarsFronius/c721a2fa
Removes Exec['reload-nginx'], because Service['nginx'] can also do the thing.

@BenoitCattie BenoitCattie merged commit 3051a92 into BenoitCattie:master Feb 29, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment