Commit Graph

22 Commits

Author SHA1 Message Date
Jeff Geerling
cf69810612 PR #46 follow-up. 2016-02-29 14:58:02 -06:00
Marcin Wolny
790badecb3 Define nginx's global options and package name. 2016-02-11 19:03:30 +01:00
Aleksandr Vinokurov
5294d77fa3 Remove new config validation, Fix #41
NOTE: current nginx config validation process breaks SELinux context
for pid file, as described in #41

Restart of nginx is not needed for configuration changes and is
replaced with reload, which will validate config before applying it,
leaving running nginx process untouched in case of any validation
errors.

Restart handler left in support.
2016-01-22 12:56:30 +03:00
Jeff Geerling
9848800982 PR #36 follow-up: clean up and reformat docs. 2015-12-28 09:57:11 -06:00
cmacrae
a248416a4d Ensuring all instances that notify 'restart nginx' also validate first 2015-12-22 16:55:29 +00:00
cmacrae
fd6b8c8e6b Addning handler for Nginx config validation:
Although the 'validate' param was added for the deployment of
/etc/nginx.conf - this validation process will only be triggered upon
changes.
So, if a vhost config is updated, but the main config isn't, the
collective config will not be verified.

I've added a new handler 'validate nginx configuration' and added this
to the 'notify' param as a first list item for vhost config changes.

Unfortunately, this will not protect against the deployment of malformed
configuration, however it will prevent the restart of Nginx in such a
situation (as the 'validate nginx configuration' handler should error
out before the 'restart nginx' handler is called).
2015-12-22 16:36:11 +00:00
cmacrae
fe55597334 Correcting indentation for 'validate' param 2015-12-22 16:18:44 +00:00
cmacrae
cc5114dc4f Re-order template deploy & add validate:
Re-ordering the tasks in this way (having the vhosts deployed first)
allows the 'validate' param to collectively check the deployed Nginx config.

Deploying vhosts after makes it hard to check their validity, as Nginx's
config checking will operate on a "master" configuration that includes
others (checking those included, also) but would error out when checking
these individual configs if they do not contain a fully working Nginx
config (which they often don't, due to their nature).
2015-12-22 16:05:28 +00:00
Asa Gage
73d7140752 updating vhost conditional expressions to check count of list members 2015-12-10 09:00:39 -05:00
Jeff Geerling
fc88ba2570 Fixes #24: Update apt cache if it hasn't been updated in a day. 2015-08-24 13:52:44 -05:00
Jeff Geerling
d1828e9c39 Fixes #13: Allow configuration of default_release parameter for apt installation. 2015-02-25 07:59:24 -06:00
Jeff Geerling
90b6ddd1b3 Fixes #11: Add default/simple virtualhost configuration. 2015-02-21 00:02:51 -06:00
Jeff Geerling
7360d9cb0b Major vhosts cleanup in preparation for issue #11. 2015-02-20 23:34:31 -06:00
Jeff Geerling
66922e9951 Fixes #10: Better organize os-specific tasks. 2015-02-20 23:15:01 -06:00
Jeff Geerling
df35be0665 Fixes #9: Use major distro version for Nginx repo on RedHat. 2015-01-23 11:07:30 -06:00
Jeff Geerling
fabb2ecbc0 Code style cleanup. 2015-01-23 10:57:06 -06:00
Jeff Geerling
fdc9a7da78 Issue #6 and PR #8: Make nginx user more flexible. 2015-01-02 11:47:26 -06:00
Jeff Geerling
736a95a012 Add option to remove default nginx vhost config. 2014-08-21 15:41:31 -05:00
Jeff Geerling
45713939ee Cleanup - This nginx role now works across RedHat and Debian. 2014-08-03 21:24:54 -05:00
Richard
9b91816d3f added support for Debian based os 2014-07-30 21:09:00 +02:00
Jeff Geerling
222479109f Update Nginx role to be more customizable. 2014-05-02 22:18:10 -05:00
Jeff Geerling
d9aebe7dee Initial commit. 2014-03-07 08:51:16 -06:00