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

Issue 29420624: Noissue - Fix nginx's restart command (Closed)

Created:
April 24, 2017, 6:53 p.m. by f.lopez
Modified:
April 25, 2017, 12:19 a.m.
Reviewers:
mathias
CC:
f.nicolaisen, Fred
Visibility:
Public.

Description

Noissue - Fix nginx's restart command

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -3 lines) Patch
M modules/nginx/manifests/init.pp View 1 chunk +1 line, -3 lines 0 comments Download

Messages

Total messages: 2
f.lopez
April 24, 2017, 6:53 p.m. (2017-04-24 18:53:44 UTC) #1
mathias
April 24, 2017, 11:14 p.m. (2017-04-24 23:14:57 UTC) #2
LGTM. But again please improve on the commit message, something like "Trigger
Nginx reload instead of restart" or similar.

Powered by Google App Engine
This is Rietveld