Commit graph

133 commits

Author SHA1 Message Date
James Fryman
32ea70c393 Merge pull request #26 from guruHub/pull_feature_confd_purge
Allow purge of confd dir as optional argument.
2012-10-02 10:37:12 -07:00
James Fryman
d10a514c6e Merge pull request #25 from guruHub/pull_refactor
puppet-nginx refactor to class/arguments
2012-10-02 10:36:54 -07:00
James Fryman
3607491270 Merge pull request #24 from guruHub/pull_issue_20
Fixed error from cat when trying nginx.d/* on nodes without vhosts defined
2012-10-02 10:36:23 -07:00
Guzmán Brasó
e9fce56092 Fixing merge from latest code on master plus guruhub 2012-10-02 01:56:37 -03:00
Guzmán Brasó
197ad06bce Implementation of new vars configtest_enable and service_restart and their respective optional arguments.
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.
2012-10-02 01:43:36 -03:00
Guzmán Brasó
e9d984eaaf Added purge support in confd.
Added confd_purge option to tell it to purge files non managed by pupet in conf.d, default is false.

Because vhost_autogen it's not actually managed by puppet but indirectly created by a puppet executed
command, it's added as ignore to avoid getting it removed.

Better formating for confd purge support

Better formating for confd purge support
2012-10-01 23:31:50 -03:00
Guzmán Brasó
e1363545ae Moved worker_connections, worker_process and proxy_set_header to the 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.
2012-10-01 22:02:44 -03:00
Guzmán Brasó
73722c0fc6 Fixed merge 2012-10-01 20:10:26 -03:00
Guzmán Brasó
af166a9ac5 Summary: Fixed error from cat when trying nginx.d/* on nodes without 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.
2012-10-01 20:04:40 -03:00
James Fryman
af4006cf66 Merge pull request #17 from lboynton/nginx-alias
Add alias support
2012-10-01 12:36:16 -07:00
James Fryman
33eb32f4f2 Merge pull request #18 from rbolkey/master
Comparison operations in nginx.conf.erb template look misplaced.
2012-10-01 12:35:43 -07:00
James Fryman
c7b2446091 Merge pull request #19 from lboynton/server-name-array
Server name array
2012-10-01 12:35:17 -07:00
James Fryman
6e843205df Merge pull request #23 from guilherme/master
Fix issue #22
2012-10-01 12:33:37 -07:00
Guilherme Reis Campos
5c4c00d726 fixing nginx.conf.erb. otherwise it will lookup for a FalseClass or TrueClass variable 2012-10-01 16:00:18 -03:00
Guzmán Brasó
c16934b472 Implementation of option configtest_enable which will call configtest 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.
2012-09-30 18:15:20 -03:00
Guzmán Brasó
7f7c7ac37d Added purge support in confd.
Added confd_purge option to tell it to purge files non managed by pupet in conf.d, default is false.
2012-09-30 07:08:21 -03:00
Guzmán Brasó
1ccca88dc2 Moved worker_connections, worker_process and proxy_set_header to the 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.
2012-09-28 14:50:16 -03:00
Guzmán Brasó
e35d314e37 Summary: Fixed error from cat when trying nginx.d/* on nodes without 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.
2012-09-28 13:00:36 -03:00
Lee Boynton
b7e7aa525d Merge remote-tracking branch 'dhutty/master' into server-name-array
Conflicts:
	manifests/resource/vhost.pp
	templates/vhost/vhost_header.erb
2012-09-27 15:42:53 +01:00
Richard Bolkey
e5b5202460 Fixing lookupvar/comparison operations in nginx.conf.erb template.
A number of comparison operations were being performed against the
parameter of the call to scope.lookupvar instead of against the result
of the call.
2012-09-24 14:21:48 -05:00
Lee Boynton
1d50d9ea05 Add alias support 2012-09-24 17:01:40 +01:00
James Fryman
3acceec622 Merge pull request #16 from lboynton/stdlib-fix
Include stdlib
2012-09-24 08:08:23 -07:00
Lee Boynton
193775263d Include stdlib rather than using class to fix conflicts with other modules using stdlib 2012-09-24 15:52:48 +01:00
James Fryman
20fa345416 Merge pull request #13 from drdla/patch-3
This is part of patch-1!
2012-09-17 10:28:15 -07:00
James Fryman
8562ff4a9a Merge pull request #12 from drdla/patch-2
This is part of patch-1!
2012-09-17 10:28:08 -07:00
James Fryman
a5b8acbd6e Merge pull request #11 from drdla/patch-1
Add parameter to rewrite www to non-www
2012-09-17 10:27:53 -07:00
James Fryman
26c6302227 Merge pull request #14 from maestrodev/master
support operatingsystem RedHat
2012-09-17 10:26:55 -07:00
Duncan Hutty
411419960b add an array parameter to resource::vhost, server_name, that goes into the server_name nginx directive for that vhost 2012-09-10 17:15:32 -04:00
Brett Porter
b9afd221a3 support platforms that don't supply lsbmajdistrelease fact 2012-09-03 11:31:40 -07:00
Brett Porter
f2a14b9961 support redhat as operating system in addition to rhel 2012-09-02 01:48:52 -07:00
Brett Porter
85ce2c42db correct class name in README 2012-09-02 00:50:57 -07:00
drdla
cc4b9615f7 Update templates/vhost/vhost_footer.erb 2012-09-01 18:52:55 +03:00
drdla
4667811324 Update templates/vhost/vhost_header.erb 2012-09-01 18:51:30 +03:00
drdla
9ca7e2b218 Add parameter to rewrite www to non-www 2012-09-01 18:46:27 +03:00
James Fryman
375dd0fa49 fixing merge conflict 2012-08-28 15:24:27 -05:00
James Fryman
3b7e191c1e fixing escape sequences 2012-08-28 15:08:30 -05:00
James Fryman
afa8586e73 Merge pull request #10 from drdla/patch-3
Fixed typo (missing , at end of line)
2012-08-28 12:53:04 -07:00
drdla
435f3a3b4f Fixed typo (missing , at end of line) 2012-08-28 22:50:18 +03:00
James Fryman
721fb2f79f Merge branch 'master' of github.com:jfryman/puppet-nginx
Conflicts:
	manifests/package/redhat.pp
2012-08-28 14:36:40 -05:00
James Fryman
08021c837c adding correct variable interpolation for nginx 2012-08-28 14:35:25 -05:00
James Fryman
6cc34f9c16 Merge pull request #9 from drdla/patch-2
removed remainder of merge conflict
2012-08-27 11:21:10 -07:00
drdla
57e82fcaa3 removed remainder of merge conflict 2012-08-27 21:18:50 +03:00
James Fryman
b9b3210a73 Merge pull request #8 from drdla/patch-1
Fixed typo (missing , at end of line)
2012-08-27 11:16:05 -07:00
drdla
6ddef7ff0e Fixed typo (missing , at end of line) 2012-08-27 21:14:44 +03:00
James Fryman
4d02012150 Merge branch 'master' of github.com:jfryman/puppet-nginx
Conflicts:
	README.markdown
	manifests/resource/location.pp
2012-08-27 09:29:46 -05:00
James Fryman
49cc046ac8 addding yumrepo for RHEL systems to ensure latest from NGINX 2012-08-27 09:26:48 -05:00
James Fryman
df8a1b5059 Merge pull request #4 from luxflux/patch-1
Fix small typo in variable name
2012-06-27 07:04:21 -07:00
Raffael Schmid
fdbccd5688 Fix small typo in variable name 2012-06-27 16:39:49 +03:00
lifton
b302fcf598 Merge pull request #21 from dukejones/master
Remove the tab characters
2012-03-04 09:36:06 -08:00
Duke Dorje
d6ac760618 Remove tab characters. 2012-03-04 09:19:44 -08:00