Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: modules/nginx/manifests/init.pp

Issue 9321094: Do not restart nginx on changes, do a reload instead (Closed)
Patch Set: Created Feb. 6, 2013, 12:04 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/nginx/manifests/init.pp
===================================================================
--- a/modules/nginx/manifests/init.pp
+++ b/modules/nginx/manifests/init.pp
@@ -48,13 +48,13 @@ class nginx (
# file {'/etc/logrotate.d/nginx':
# source => 'nginx/logrotate',
# require => File['/etc/nginx/nginx.conf']
# }
service {'nginx':
ensure => running,
enable => true,
- hasrestart => true,
+ restart => '/etc/init.d/nginx reload',
hasstatus => true,
require => File['/etc/nginx/nginx.conf']
}
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld