jfryman/puppet-nginx — pull request #27 by guzmanbraso (2012-10-02 05:04:31+00:00)
Title: Implementation of new vars configtest_enable and service_restart...

Argument configtest_enable / params.pp $nx_configtest_enable
* Default false
* If true will set service[nginx] restart with contents of nx_service_restart.

Argument service_restart / params.pp $nx_service_restart
* Default '/etc/init.d/nginx configtest && /etc/init.d/nginx restart'
* Since nginx 0.7.53 nginx supports '-s HUP' which will reload testing configuration first, to be backwards compatible above default was choosen.

Many distributions of nginx already implement a configtest before restart, however many doesn't, and many even don't provide restart but a stop/start combination. If configtest_enable is true then puppet will force nginx to do a configtest no matter if it was going or not to do it itself.