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

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

Issue 29420624: Noissue - Fix nginx's restart command (Closed)
Patch Set: Created April 24, 2017, 6:53 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
@@ -222,9 +222,7 @@
$restart_command = join([
'set -e',
'pid=`cat /var/run/nginx.pid`',
- 'kill -USR2 "$pid"',
- 'sleep 2',
- 'kill -QUIT "$pid"',
+ 'kill -HUP "$pid"',
], "\n")
service {'nginx':
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld