Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature status locations #28

Merged
merged 8 commits into from
Oct 3, 2012

Conversation

guzmanbraso
Copy link
Contributor

Add support to create locations for module stub_status

Useful to gather data from monitoring systems like nagios/zabbix.

…vhosts defined

Description:
When running first time from scratch, if class was included without any setup at all
and with sane defaults, in debian squeeze generates an error when trying to collect
non existant vhosts, because cat complains that can not expand * because there are no
files.

Changed:
Added unless to check that at least there is a file in nginx.d by testing it can expand.
…new structure, backwards compatible

Defaults are set inside params, nginx class will set default and send it from local var to nginx::config,
so even when there is no need for set default values on nginx::config, in case someone already using
this module it's for some reason calling directly nginx::config, to avoid breaking anything defaults are
set inside nginx::config too.
Added confd_purge option to tell it to purge files non managed by pupet in conf.d, default is false.
… on each

service restart, therefore if user submit an invalid configuration nginx won't
stop and realize too late that it can start again.

By default it's false, if true it assumes you will have a working /etc/init.d/nginx
which accepts configtest and restart.
If stub_status is provided as true a location will be created to provide
nginx status information from stub_status module.
jfryman added a commit that referenced this pull request Oct 3, 2012
@jfryman jfryman merged commit c63df59 into voxpupuli:master Oct 3, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants